commit 4b10f70918dc565a8f7a755b555904d9cb029bd0
author cromwellian <cromwellian@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Thu Aug 13 19:24:26 2009 +0000
committer cromwellian <cromwellian@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Thu Aug 13 19:24:26 2009 +0000
tree e65f5b46751701d021944e4d27d01c323ed1e453
parent 6b2986d76951a402ed8e8f555dbf2a93ed5be568

Added unary prefix '+' operator to AST to fix, Issue #3942. Modified Js code-gen to insert spaces in appropriate places to maintain correct precedence. (e.g. a+ +b should not become a++b)






git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@5960 8db76d5a-ed1c-0410-87a9-c151d255dfc7

dev/core/src/com/google/gwt/dev/js/JsParser.java

diff --git a/dev/core/src/com/google/gwt/dev/js/JsParser.java b/dev/core/src/com/google/gwt/dev/js/JsParser.java
index 7fdde5b..49fb876 100644
--- a/dev/core/src/com/google/gwt/dev/js/JsParser.java
+++ b/dev/core/src/com/google/gwt/dev/js/JsParser.java
@@ -1197,8 +1197,13 @@
         return mapPrefixOperation(JsUnaryOperator.TYPEOF, unOp);
 
       case TokenStream.ADD:
-        // Pretend we didn't see it.
-        return mapExpression(unOp.getFirstChild());
+        if (unOp.getFirstChild().getType() != TokenStream.NUMBER) {
+          return mapPrefixOperation(JsUnaryOperator.POS, unOp);
+        } else {
+          // Pretend we didn't see it.
+          return mapExpression(unOp.getFirstChild());
+        }
+
       case TokenStream.VOID:
         return mapPrefixOperation(JsUnaryOperator.VOID, unOp);
 

dev/core/src/com/google/gwt/dev/js/JsToStringGenerationVisitor.java

diff --git a/dev/core/src/com/google/gwt/dev/js/JsToStringGenerationVisitor.java b/dev/core/src/com/google/gwt/dev/js/JsToStringGenerationVisitor.java
index 78dc37c..7cd109a 100644
--- a/dev/core/src/com/google/gwt/dev/js/JsToStringGenerationVisitor.java
+++ b/dev/core/src/com/google/gwt/dev/js/JsToStringGenerationVisitor.java
@@ -1188,7 +1188,8 @@
       JsOperator op2 = ((JsPrefixOperation) arg).getOperator();
       return (op == JsBinaryOperator.SUB || op == JsUnaryOperator.NEG)
           && (op2 == JsUnaryOperator.DEC || op2 == JsUnaryOperator.NEG)
-          || (op == JsBinaryOperator.ADD && op2 == JsUnaryOperator.INC);
+          || (op == JsBinaryOperator.ADD || op == JsUnaryOperator.POS)
+          && (op2 == JsUnaryOperator.INC || op2 == JsUnaryOperator.POS);
     }
     if (arg instanceof JsNumberLiteral) {
       JsNumberLiteral literal = (JsNumberLiteral) arg;