commit b5768af15f2949385b3d782199336d70e56e00e2
author fzaiser <fzaiser@google.com> Mon Oct 09 15:16:50 2017 +0200
committer Marcel Hlopko <hlopko@google.com> Tue Oct 10 11:22:04 2017 +0200
tree ac2eb5c63147cd8a27933f528f4475fcd771f94f
parent 8880b7abceb78b22b56be53d02bf97d9e8cb74cb

Fix the location of load statements in Skylark.

Previously, the closing parenthesis was not included in the location.

RELNOTES: none
PiperOrigin-RevId: 171527078

src/main/java/com/google/devtools/build/lib/syntax/Parser.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
index 6d45726..e59f329 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
@@ -1053,10 +1053,10 @@
 
       parseLoadSymbol(symbols);
     }
-    expect(TokenKind.RPAREN);
 
     LoadStatement stmt = new LoadStatement(importString, symbols);
-    list.add(setLocation(stmt, start, token.left));
+    list.add(setLocation(stmt, start, token.right));
+    expect(TokenKind.RPAREN);
     expectAndRecover(TokenKind.NEWLINE);
   }