diff options
author | fzaiser <fzaiser@google.com> | 2017-10-24 15:37:50 +0200 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2017-10-24 17:25:50 +0200 |
commit | 1a92d56311d4680016b4d6743579fa19b1c32d9f (patch) | |
tree | b5cfcaa467210a77aa4eb793be716195448009aa /src/tools/skylark | |
parent | e05c84fdfd2a5ae80a8cc0a01c7460064ccfbbf8 (diff) |
Parser: fix incorrect end location for statement suites.
This used to be annoying for Skylint.
RELNOTES: none
PiperOrigin-RevId: 173249428
Diffstat (limited to 'src/tools/skylark')
-rw-r--r-- | src/tools/skylark/javatests/com/google/devtools/skylark/skylint/ControlFlowCheckerTest.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/ControlFlowCheckerTest.java b/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/ControlFlowCheckerTest.java index c522dcbee4..e104d5d508 100644 --- a/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/ControlFlowCheckerTest.java +++ b/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/ControlFlowCheckerTest.java @@ -68,7 +68,7 @@ public class ControlFlowCheckerTest { .toString(); Truth.assertThat(messages) .contains( - "1:1-5:26: some but not all execution paths of 'some_function' return a value" + "1:1-5:10: some but not all execution paths of 'some_function' return a value" + " [missing-return-value]"); Truth.assertThat(messages) .contains( @@ -137,7 +137,7 @@ public class ControlFlowCheckerTest { " # return missing here") .toString()) .contains( - "1:1-5:23: some but not all execution paths of 'some_function' return a value" + "1:1-4:14: some but not all execution paths of 'some_function' return a value" + " [missing-return-value]"); } @@ -152,7 +152,7 @@ public class ControlFlowCheckerTest { " # return missing here") .toString()) .contains( - "1:1-5:23: some but not all execution paths of 'some_function' return a value" + "1:1-4:14: some but not all execution paths of 'some_function' return a value" + " [missing-return-value]"); } |