commit 7aa2c8ee90427212b1ef6a2329602557e2d134a8
author laurentlb <laurentlb@google.com> Thu Oct 18 10:09:30 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Thu Oct 18 10:12:00 2018 -0700
tree 351951859e8d6a9d4d5938d5715645d46962f7b9
parent 98dbfea78686488acc215cc82008842c3d1a2a2c

Parser: Allow `not not x`

The bug was first found and fixed in the Go interpreter (https://github.com/google/skylark/pull/129).

BUG=117284860

RELNOTES: None.
PiperOrigin-RevId: 217721589

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 9656ea1..9370cfb 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
@@ -998,7 +998,7 @@
   private Expression parseNotExpression(int prec) {
     int start = token.left;
     expect(TokenKind.NOT);
-    Expression expression = parseNonTupleExpression(prec + 1);
+    Expression expression = parseNonTupleExpression(prec);
     UnaryOperatorExpression notExpression =
         new UnaryOperatorExpression(UnaryOperator.NOT, expression);
     return setLocation(notExpression, start, expression);