This is an automated email from the ASF dual-hosted git repository. matthiasblaesing pushed a commit to branch graaljs2 in repository https://gitbox.apache.org/repos/asf/netbeans.git
commit e24b95184fc5d7fee3964b5d4588819047ba582a Author: Matthias Bläsing <mblaes...@doppel-helix.eu> AuthorDate: Fri May 3 20:34:42 2019 +0200 Fix compile errors --- .../editor/formatter/JsFormatVisitor.java | 2 +- .../javascript2/editor/hints/Ecma6Rule.java | 9 +- .../javascript2/editor/hints/Ecma7Rule.java | 2 +- .../javascript2/editor/hints/JsConventionRule.java | 10 +-- .../javascript2/editor/hints/WeirdAssignment.java | 2 +- .../javascript2/editor/parser/JsParser.java | 2 +- .../javascript2/editor/parser/AstXmlVisitor.java | 6 +- .../modules/javascript2/model/ModelVisitor.java | 95 +++++++++++----------- .../javascript2/model/SemiTypeResolverVisitor.java | 26 +++--- 9 files changed, 75 insertions(+), 79 deletions(-) diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/formatter/JsFormatVisitor.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/formatter/JsFormatVisitor.java index 1fcedd3..630ef37 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/formatter/JsFormatVisitor.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/formatter/JsFormatVisitor.java @@ -1450,7 +1450,7 @@ public class JsFormatVisitor extends NodeVisitor implements JsxNodeVisitor { } private int getStart(BinaryNode node) { - return getStart(node.lhs()); + return getStart(node.getLhs()); } private static int getFunctionStart(FunctionNode node) { diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma6Rule.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma6Rule.java index 290d1f6..281ad89 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma6Rule.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma6Rule.java @@ -35,7 +35,6 @@ import com.oracle.js.parser.ir.Statement; import com.oracle.js.parser.ir.TernaryNode; import com.oracle.js.parser.ir.UnaryNode; import com.oracle.js.parser.ir.VarNode; -import com.sun.javafx.fxml.expression.BinaryExpression; import java.util.Arrays; import java.util.Collections; import java.util.List; @@ -169,17 +168,17 @@ public class Ecma6Rule extends EcmaLevelRule { } BinaryNode bn = (BinaryNode)tn.getTest(); if (!( - (bn.lhs() instanceof ParameterNode) && - (bn.rhs() instanceof UnaryNode) && + (bn.getLhs() instanceof ParameterNode) && + (bn.getRhs() instanceof UnaryNode) && (bn.tokenType() == TokenType.EQ_STRICT) && - (bn.rhs().tokenType() == TokenType.VOID))) { + (bn.getRhs().tokenType() == TokenType.VOID))) { return false; } if (!((tn.getTrueExpression() instanceof JoinPredecessorExpression) && (tn.getFalseExpression() instanceof JoinPredecessorExpression))) { return false; } Expression e = ((JoinPredecessorExpression)tn.getFalseExpression()).getExpression(); - if (e != bn.lhs()) { + if (e != bn.getLhs()) { return false; } return true; diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma7Rule.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma7Rule.java index 8d0356a..3bdc40d 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma7Rule.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/Ecma7Rule.java @@ -185,7 +185,7 @@ public class Ecma7Rule extends EcmaLevelRule { int position = Token.descPosition(token); addHint(context, hints, new OffsetRange(position, position + Token.descLength(token))); } else if (TokenType.COMMARIGHT == type) { - Expression rhs = binaryNode.rhs(); + Expression rhs = binaryNode.getRhs(); if (!rhs.isTokenType(TokenType.COMMARIGHT)) { checkTrailingComma(rhs.getFinish()); } diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/JsConventionRule.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/JsConventionRule.java index ce9af1b..96d4910 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/JsConventionRule.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/JsConventionRule.java @@ -301,11 +301,11 @@ public class JsConventionRule extends JsAstRule { ModelUtils.documentOffsetRange(context.getJsParserResult(), condition.getStart(), condition.getFinish()), null, 500)); } } - if (binaryNode.lhs() instanceof BinaryNode) { - checkAssignmentInCondition(binaryNode.lhs()); + if (binaryNode.getLhs() instanceof BinaryNode) { + checkAssignmentInCondition(binaryNode.getLhs()); } - if (binaryNode.rhs() instanceof BinaryNode) { - checkAssignmentInCondition(binaryNode.rhs()); + if (binaryNode.getRhs() instanceof BinaryNode) { + checkAssignmentInCondition(binaryNode.getRhs()); } } } @@ -513,7 +513,7 @@ public class JsConventionRule extends JsAstRule { if (arrayTrailingComma != null) { if (literalNode.getValue() instanceof Node[]) { Node previous = getPath().get(getPath().size() - 1); - if (previous instanceof BinaryNode && ((BinaryNode) previous).lhs() == literalNode) { + if (previous instanceof BinaryNode && ((BinaryNode) previous).getLhs() == literalNode) { // destructuring assignment return super.enterLiteralNode(literalNode); } diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/WeirdAssignment.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/WeirdAssignment.java index e39a45a..72142b3 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/WeirdAssignment.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/hints/WeirdAssignment.java @@ -86,7 +86,7 @@ public class WeirdAssignment extends JsAstRule { @Override public boolean enterBinaryNode(BinaryNode binaryNode) { - if (binaryNode.isTokenType(TokenType.ASSIGN) && binaryNode.lhs().toString().equals(binaryNode.rhs().toString())) { + if (binaryNode.isTokenType(TokenType.ASSIGN) && binaryNode.getLhs().toString().equals(binaryNode.getRhs().toString())) { hints.add(new Hint(rule, Bundle.JsWeirdAssignmentDN(), context.getJsParserResult().getSnapshot().getSource().getFileObject(), ModelUtils.documentOffsetRange(context.getJsParserResult(), binaryNode.getStart(), binaryNode.getFinish()), null, 500)); diff --git a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/parser/JsParser.java b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/parser/JsParser.java index 54b8360..5487856 100644 --- a/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/parser/JsParser.java +++ b/webcommon/javascript2.editor/src/org/netbeans/modules/javascript2/editor/parser/JsParser.java @@ -96,7 +96,7 @@ public class JsParser extends SanitizingParser<JsParserResult> { ScriptEnvironment.Builder builder = ScriptEnvironment.builder(); Parser parser = new Parser( - builder.emptyStatements(true).es6(true)/* .jsx (true) */.build(), + builder.emptyStatements(true).build(), source, errorManager); FunctionNode node = null; diff --git a/webcommon/javascript2.editor/test/unit/src/org/netbeans/modules/javascript2/editor/parser/AstXmlVisitor.java b/webcommon/javascript2.editor/test/unit/src/org/netbeans/modules/javascript2/editor/parser/AstXmlVisitor.java index 262c9f7..daa7bdb 100644 --- a/webcommon/javascript2.editor/test/unit/src/org/netbeans/modules/javascript2/editor/parser/AstXmlVisitor.java +++ b/webcommon/javascript2.editor/test/unit/src/org/netbeans/modules/javascript2/editor/parser/AstXmlVisitor.java @@ -258,8 +258,8 @@ public class AstXmlVisitor extends NodeVisitor implements JsxNodeVisitor { processAttribute(node.isSelfModifying(), "isSelfModifying"); // processWithComment(node.getAssignmentDest(), "BinaryNode AssignmentDest"); // processWithComment(node.getAssignmentSource(), "BinaryNode AssignmentSource"); - processWithComment(node.lhs(), "BinaryNode lhs"); - processWithComment(node.rhs(), "BinaryNode rhs"); + processWithComment(node.getLhs(), "BinaryNode lhs"); + processWithComment(node.getRhs(), "BinaryNode rhs"); createCloseTag(node); return false; } @@ -329,7 +329,6 @@ public class AstXmlVisitor extends NodeVisitor implements JsxNodeVisitor { processAttribute(node.isProgram(), "isProgram"); processAttribute(node.isNamedFunctionExpression(), "isNamedFunctionExpression"); processAttribute(node.isSubclassConstructor(), "isSubclassConstructor"); - processAttribute(node.isVarArg(), "isVarArg"); processAttribute(node.isAsync(), "isAsync"); processAttribute(node); @@ -367,7 +366,6 @@ public class AstXmlVisitor extends NodeVisitor implements JsxNodeVisitor { // processAttribute(node.isDestructuredParameter(), "isDestructuredParameter"); processAttribute(node.isDirectSuper(), "isDirectSuppert"); processAttribute(node.isFunction(), "isFunction"); - processAttribute(node.isFutureStrictName(), "isFutureStrictName"); processAttribute(node.isInitializedHere(), "isInitializedHere"); processAttribute(node.isInternal(), "isInternal"); processAttribute(node.isPropertyName(), "isPropertyName"); diff --git a/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/ModelVisitor.java b/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/ModelVisitor.java index e82a52c..b01d439 100644 --- a/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/ModelVisitor.java +++ b/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/ModelVisitor.java @@ -254,8 +254,8 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { @Override public boolean enterBinaryNode(BinaryNode binaryNode) { - Node lhs = binaryNode.lhs(); - Node rhs = binaryNode.rhs(); + Node lhs = binaryNode.getLhs(); + Node rhs = binaryNode.getRhs(); if (lhs instanceof LiteralNode.ArrayLiteralNode && binaryNode.tokenType() == TokenType.ASSIGN) { // case of destructuring assgnment like [a,b] = .... LiteralNode.ArrayLiteralNode lan = (LiteralNode.ArrayLiteralNode)lhs; @@ -320,9 +320,9 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { variableName = ((IdentNode)lPropertyNode.getValue()).getName(); } else if (lPropertyNode.getValue() instanceof BinaryNode) { BinaryNode bNode = (BinaryNode)lPropertyNode.getValue(); - if (bNode.tokenType() == TokenType.ASSIGN && bNode.lhs() instanceof IdentNode) { + if (bNode.tokenType() == TokenType.ASSIGN && bNode.getLhs() instanceof IdentNode) { // the default parameter {a=10, b=20} = .... - variableName = ((IdentNode)bNode.lhs()).getName(); + variableName = ((IdentNode)bNode.getLhs()).getName(); } } if (variableName != null) { @@ -355,9 +355,9 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { return key.getName().equals(value.getName()) && key.getStart() == value.getStart(); } if (pNode.getKey() instanceof IdentNode && pNode.getValue() instanceof BinaryNode - && ((BinaryNode)pNode.getValue()).tokenType() == TokenType.ASSIGN && ((BinaryNode)pNode.getValue()).lhs() instanceof IdentNode) { + && ((BinaryNode)pNode.getValue()).tokenType() == TokenType.ASSIGN && ((BinaryNode)pNode.getValue()).getLhs() instanceof IdentNode) { IdentNode key = (IdentNode)pNode.getKey(); - IdentNode value = (IdentNode)((BinaryNode)pNode.getValue()).lhs(); + IdentNode value = (IdentNode)((BinaryNode)pNode.getValue()).getLhs(); return key.getName().equals(value.getName()) && key.getStart() == value.getStart(); } return false; @@ -493,10 +493,10 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { @Override public Node leaveBinaryNode(BinaryNode binaryNode) { - Node lhs = binaryNode.lhs(); - Node rhs = binaryNode.rhs(); + Node lhs = binaryNode.getLhs(); + Node rhs = binaryNode.getRhs(); if (lhs instanceof IdentNode && rhs instanceof BinaryNode) { - Node rlhs = ((BinaryNode)rhs).lhs(); + Node rlhs = ((BinaryNode)rhs).getLhs(); if (rlhs instanceof IdentNode) { JsObject origFunction = modelBuilder.getCurrentDeclarationFunction().getProperty(((IdentNode)rlhs).getName()); if (origFunction != null && origFunction.getJSKind().isFunction()) { @@ -1655,23 +1655,23 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { } else if (init instanceof BinaryNode) { BinaryNode bNode = (BinaryNode) init; if (bNode.isLogical() - && ((bNode.rhs() instanceof JoinPredecessorExpression && ((JoinPredecessorExpression) bNode.rhs()).getExpression() instanceof FunctionNode) - || (bNode.lhs() instanceof JoinPredecessorExpression && ((JoinPredecessorExpression) bNode.lhs()).getExpression() instanceof FunctionNode))) { + && ((bNode.getRhs() instanceof JoinPredecessorExpression && ((JoinPredecessorExpression) bNode.getRhs()).getExpression() instanceof FunctionNode) + || (bNode.getLhs() instanceof JoinPredecessorExpression && ((JoinPredecessorExpression) bNode.getLhs()).getExpression() instanceof FunctionNode))) { // case: var f1 = xxx || function () {} // the function here is already, need to be just fixed the name offsets createVariable = false; } else if (bNode.isAssignment()) { createVariable = false; if (parentFn.getProperty(varNode.getName().getName()) == null) { - while (bNode.rhs() instanceof BinaryNode && bNode.rhs().isAssignment()) { + while (bNode.getRhs() instanceof BinaryNode && bNode.getRhs().isAssignment()) { // the cycle is trying to find out a FunctionNode at the end of assignements // case var f1 = f2 = f3 = f4 = function () {} - bNode = (BinaryNode) bNode.rhs(); + bNode = (BinaryNode) bNode.getRhs(); } - if (bNode.rhs() instanceof FunctionNode) { + if (bNode.getRhs() instanceof FunctionNode) { // case var f1 = f2 = function (){}; // -> the variable will be reference fo the function - FunctionNode fNode = (FunctionNode) bNode.rhs(); + FunctionNode fNode = (FunctionNode) bNode.getRhs(); JsObject original = parentFn.getProperty(modelBuilder.getFunctionName(fNode)); if (original != null) { Identifier varName = new Identifier(varNode.getName().getName(), getOffsetRange(varNode.getName())); @@ -1800,8 +1800,8 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { isDeclaredInParent = true; } else if (lastVisited instanceof BinaryNode) { BinaryNode binNode = (BinaryNode) lastVisited; - if (binNode.lhs() instanceof IndexNode) { - Node index = ((IndexNode)binNode.lhs()).getIndex(); + if (binNode.getLhs() instanceof IndexNode) { + Node index = ((IndexNode)binNode.getLhs()).getIndex(); if (!(index instanceof LiteralNode && ((LiteralNode)index).isString())) { treatAsAnonymous = true; } @@ -1811,10 +1811,10 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { lastVisited = getPath().get(getPath().size() - pathIndex - 1); } fqName = getName(binNode, parserResult); - if ((binNode.lhs() instanceof IdentNode) - || (binNode.lhs() instanceof AccessNode - && ((AccessNode) binNode.lhs()).getBase() instanceof IdentNode - && ((IdentNode) ((AccessNode) binNode.lhs()).getBase()).getName().equals(ModelUtils.THIS))) { //NOI18N + if ((binNode.getLhs() instanceof IdentNode) + || (binNode.getLhs() instanceof AccessNode + && ((AccessNode) binNode.getLhs()).getBase() instanceof IdentNode + && ((IdentNode) ((AccessNode) binNode.getLhs()).getBase()).getName().equals(ModelUtils.THIS))) { //NOI18N if (lastVisited instanceof ExpressionStatement && !fqName.get(0).getName().equals(ModelUtils.THIS)) { //NOI18N // try to catch the case: pool = []; List<Identifier> objectName = fqName.size() > 1 ? fqName.subList(0, fqName.size() - 1) : fqName; @@ -1825,7 +1825,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { } } else { isDeclaredInParent = true; - if (!(binNode.lhs() instanceof IdentNode)) { + if (!(binNode.getLhs() instanceof IdentNode)) { parent = resolveThis(modelBuilder.getCurrentObject()); } } @@ -1952,11 +1952,11 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { } if (lastVisited instanceof BinaryNode) { BinaryNode bNode = (BinaryNode)lastVisited; - if (bNode.lhs().equals(objectNode)) { + if (bNode.getLhs().equals(objectNode)) { // case of destructuring assignment { a, b} = .... // we should not create object in the model. return super.enterObjectNode(objectNode); - } else if (bNode.lhs() instanceof ObjectNode && bNode.rhs().equals(objectNode)) { + } else if (bNode.getLhs() instanceof ObjectNode && bNode.getRhs().equals(objectNode)) { // case of destructuring assignment {a, b} = {a:1, b:2} // do nothing/ already processed in binary node. return false; @@ -1984,7 +1984,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { treatAsAnonymous = true; } else if (lastVisited instanceof BinaryNode) { BinaryNode binNode = (BinaryNode) lastVisited; - Node binLhs = binNode.lhs(); + Node binLhs = binNode.getLhs(); if (binLhs instanceof IndexNode) { Node index = ((IndexNode)binLhs).getIndex(); if (!(index instanceof LiteralNode && ((LiteralNode)index).isString())) { @@ -2083,7 +2083,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { if (lastVisited instanceof BinaryNode) { BinaryNode bNode = (BinaryNode)lastVisited; - if (bNode.lhs().equals(objectNode)) { + if (bNode.getLhs().equals(objectNode)) { // case of destructuring assignment { a, b} = .... // we dob't create object in the model, but process the property nodes -> skip reseting modelBuilder return super.leaveObjectNode(objectNode); @@ -2165,7 +2165,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { boolean isDestructiveParam = false; if (node instanceof Block) { Block block = (Block)node; - if (block.isParameterBlock() && oNode != null && bNode != null && bNode.lhs().equals(oNode) ) { + if (block.isParameterBlock() && oNode != null && bNode != null && bNode.getLhs().equals(oNode) ) { // we are in parameters defined via destructive assignment JsFunction currentFnImpl = modelBuilder.getCurrentDeclarationFunction(); property = (JsObjectImpl)currentFnImpl.getParameter(name.getName()); @@ -2236,7 +2236,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { // Node lastNode = getPreviousFromPath(1); // if (!((lastNode instanceof VarNode) && !reference.isAnonymous())) { // if (lastNode instanceof BinaryNode && !reference.isAnonymous()) { -// Node lhs = ((BinaryNode)lastNode).lhs(); +// Node lhs = ((BinaryNode)lastNode).getLhs(); // List<Identifier> nodeName = getNodeName(lhs, parserResult); // if (nodeName != null && !nodeName.isEmpty()) { // JsObject jsObject = null; @@ -2350,11 +2350,11 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { // this should handle cases like // var prom = another.prom = function prom() {} BinaryNode bNode = (BinaryNode)init; - while (bNode.rhs() instanceof BinaryNode ) { - bNode = (BinaryNode)bNode.rhs(); + while (bNode.getRhs() instanceof BinaryNode ) { + bNode = (BinaryNode)bNode.getRhs(); } - if (bNode.rhs() instanceof FunctionNode) { - rNode = (FunctionNode) bNode.rhs(); + if (bNode.getRhs() instanceof FunctionNode) { + rNode = (FunctionNode) bNode.getRhs(); } } else if (init instanceof UnaryNode && ((UnaryNode)init).getExpression() instanceof CallNode && ((CallNode)((UnaryNode)init).getExpression()).getFunction() instanceof FunctionNode) { @@ -2533,14 +2533,14 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { // this should handle cases like // var prom = another.prom = function prom() {} BinaryNode bNode = (BinaryNode)init; - while (bNode.rhs() instanceof BinaryNode ) { - bNode = (BinaryNode)bNode.rhs(); + while (bNode.getRhs() instanceof BinaryNode ) { + bNode = (BinaryNode)bNode.getRhs(); } - if (bNode.rhs() instanceof FunctionNode) { + if (bNode.getRhs() instanceof FunctionNode) { // this should handle cases like // var prom = another.prom = function prom() {} - rNode = (FunctionNode) bNode.rhs(); - List<Identifier> name = getNodeName(bNode.lhs(), parserResult); + rNode = (FunctionNode) bNode.getRhs(); + List<Identifier> name = getNodeName(bNode.getLhs(), parserResult); if (name != null && !name.isEmpty()) { boolean isPriviliged = name.get(0).getName().equals(ModelUtils.THIS); JsObject parent = isPriviliged ? resolveThis(modelBuilder.getCurrentObject()) : modelBuilder.getCurrentObject(); @@ -2564,8 +2564,8 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { } } -// if (bNode.rhs() instanceof ReferenceNode /*&& bNode.tokenType() == TokenType.ASSIGN*/) { -// init = (ReferenceNode) bNode.rhs(); +// if (bNode.getRhs() instanceof ReferenceNode /*&& bNode.tokenType() == TokenType.ASSIGN*/) { +// init = (ReferenceNode) bNode.getRhs(); // } } else if (init instanceof FunctionNode) { rNode = (FunctionNode)init; @@ -2712,7 +2712,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { private static List<Identifier> getName(BinaryNode binaryNode, ParserResult parserResult) { List<Identifier> name = new ArrayList(); - Node lhs = binaryNode.lhs(); + Node lhs = binaryNode.getLhs(); if (lhs instanceof AccessNode) { name = getName((AccessNode)lhs, parserResult); } else if (lhs instanceof IdentNode) { @@ -2779,7 +2779,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { } else if (base instanceof LiteralNode) { // we fake Number object to handel mark occurrences LiteralNode lNode= (LiteralNode) base; - if (lNode.isNumeric()) { + if (lNode.getValue() instanceof Number) { baseIdent = new Identifier("Number", OffsetRange.NONE); //NOI8N } } @@ -2817,7 +2817,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { Node lastNode = getPreviousFromPath(2); if (lastNode instanceof BinaryNode) { BinaryNode bNode = (BinaryNode) lastNode; - if (bNode.lhs().equals(accessNode)) { + if (bNode.getLhs().equals(accessNode)) { object = (JsObjectImpl) variable; } } @@ -2886,7 +2886,7 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { boolean onLeftSite = false; if (lastVisited instanceof BinaryNode) { BinaryNode bNode = (BinaryNode)lastVisited; - onLeftSite = bNode.tokenType() == TokenType.ASSIGN && bNode.lhs().equals(accessNode); + onLeftSite = bNode.tokenType() == TokenType.ASSIGN && bNode.getLhs().equals(accessNode); } String propertyName = accessNode.getProperty(); // there is a problem in the parser. When there is a line comment after access node, then the finish of access node is finish of the line comment @@ -3335,8 +3335,8 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { if (getPreviousFromPath(pathIndex + 3) instanceof BinaryNode) { BinaryNode bNode = (BinaryNode) getPreviousFromPath(pathIndex + 3); if (bNode.tokenType() == TokenType.ASSIGN) { - if (bNode.lhs() instanceof AccessNode) { - List<Identifier> identifier = getName((AccessNode) bNode.lhs(), parserResult); + if (bNode.getLhs() instanceof AccessNode) { + List<Identifier> identifier = getName((AccessNode) bNode.getLhs(), parserResult); if (identifier != null) { if (!identifier.isEmpty() && ModelUtils.THIS.equals(identifier.get(0).getName())) { identifier.remove(0); @@ -3352,8 +3352,8 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { simpleName = false; } } - } else if (bNode.lhs() instanceof IdentNode) { - name = ((IdentNode) bNode.lhs()).getName(); + } else if (bNode.getLhs() instanceof IdentNode) { + name = ((IdentNode) bNode.getLhs()).getName(); } } } else if (getPreviousFromPath(pathIndex + 3) instanceof VarNode) { @@ -3547,7 +3547,6 @@ public class ModelVisitor extends PathNodeVisitor implements ModelResolver { if (fn.isDeclared()) sb.append(", isDeclared"); if (fn.isMethod()) sb.append(", isMethod"); if (fn.isNamedFunctionExpression()) sb.append(", isNamedFunctionExpression"); - if (fn.isVarArg()) sb.append(", isVarArg"); if (fn.getFlag(FunctionNode.HAS_FUNCTION_DECLARATIONS)) sb.append(", hasDeclaredFunctions"); if (fn.hasDirectSuper()) sb.append(", hasDirectSuper"); // if (fn.hasScopeBlock()) sb.append(", hasScoprBlock"); diff --git a/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/SemiTypeResolverVisitor.java b/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/SemiTypeResolverVisitor.java index a4efcf2..5cb3a56 100644 --- a/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/SemiTypeResolverVisitor.java +++ b/webcommon/javascript2.model/src/org/netbeans/modules/javascript2/model/SemiTypeResolverVisitor.java @@ -359,20 +359,20 @@ public class SemiTypeResolverVisitor extends PathNodeVisitor { } return false; } - binaryNode.lhs().accept(this); - add(exp, offsetVisitor.findOffset(binaryNode.lhs()), false); + binaryNode.getLhs().accept(this); + add(exp, offsetVisitor.findOffset(binaryNode.getLhs()), false); reset(); - binaryNode.rhs().accept(this); - add(exp, offsetVisitor.findOffset(binaryNode.rhs()), false); + binaryNode.getRhs().accept(this); + add(exp, offsetVisitor.findOffset(binaryNode.getRhs()), false); reset(); return false; } - if (binaryNode.rhs() instanceof FunctionNode) { - binaryNode.lhs().accept(this); + if (binaryNode.getRhs() instanceof FunctionNode) { + binaryNode.getLhs().accept(this); return false; } if (binaryNode.isAssignment()) { - binaryNode.rhs().accept(this); + binaryNode.getRhs().accept(this); return false; } return super.enterBinaryNode(binaryNode); @@ -395,8 +395,8 @@ public class SemiTypeResolverVisitor extends PathNodeVisitor { private boolean isResultString(BinaryNode binaryNode) { boolean bResult = false; TokenType tokenType = binaryNode.tokenType(); - Node lhs = binaryNode.lhs(); - Node rhs = binaryNode.rhs(); + Node lhs = binaryNode.getLhs(); + Node rhs = binaryNode.getRhs(); if (tokenType == TokenType.ADD && ((lhs instanceof LiteralNode && ((LiteralNode) lhs).isString()) || (rhs instanceof LiteralNode && ((LiteralNode) rhs).isString()))) { @@ -420,11 +420,11 @@ public class SemiTypeResolverVisitor extends PathNodeVisitor { private boolean isResultNumber(BinaryNode binaryNode) { boolean bResult = false; TokenType tokenType = binaryNode.tokenType(); - Node lhs = binaryNode.lhs(); - Node rhs = binaryNode.rhs(); + Node lhs = binaryNode.getLhs(); + Node rhs = binaryNode.getRhs(); if ((tokenType == TokenType.BIT_OR || tokenType == TokenType.BIT_AND) - && ((lhs instanceof LiteralNode && ((LiteralNode) lhs).isNumeric()) - || (rhs instanceof LiteralNode && ((LiteralNode) rhs).isNumeric()))) { + && ((lhs instanceof LiteralNode && ((LiteralNode) lhs).getValue() instanceof Number) + || (rhs instanceof LiteralNode && ((LiteralNode) rhs).getValue() instanceof Number))) { bResult = true; } else if (tokenType == TokenType.DIV || tokenType == TokenType.MUL || tokenType == TokenType.SUB ){ --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@netbeans.apache.org For additional commands, e-mail: commits-h...@netbeans.apache.org For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists