Never let JSDoc from one statement spill into the next statement.

See http://code.google.com/p/closure-compiler/issues/detail?id=428 for more info Committed by Nick Santos ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=26339541
travisennis · May 9, 2012 · af4a9af · af4a9af
1 parent 2bb5493
commit af4a9af
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/org/mozilla/javascript/Parser.java b/src/org/mozilla/javascript/Parser.java
@@ -2075,7 +2075,7 @@ private AstNode assignExpr()
             if (jsdocNode != null) {
                 pn.setJsDocNode(jsdocNode);
             }
-        } else if (tt == Token.SEMI && pn.getType() == Token.GETPROP) {
+        } else if (tt == Token.SEMI) {
             // This may be dead code added intentionally, for JSDoc purposes.
             // For example: /** @type Number */ C.prototype.x;
             if (currentJsDocComment != null) {

diff --git a/testsrc/org/mozilla/javascript/tests/ParserTest.java b/testsrc/org/mozilla/javascript/tests/ParserTest.java
@@ -931,6 +931,15 @@ public void testJSDocAttachment9() {
                        .getExpression().getJsDoc());
     }
 
+    public void testJSDocAttachment15() {
+        AstRoot root = parse("/** @private */ x(); function f() {}");
+        assertNotNull(root.getComments());
+        assertEquals(1, root.getComments().size());
+
+        ExpressionStatement st = (ExpressionStatement) root.getFirstChild();
+        assertNotNull(st.getExpression().getJsDoc());
+    }
+
     public void testParsingWithoutJSDoc() {
         AstRoot root = parse("var a = /** @type number */(x);", false);
         assertNotNull(root.getComments());