From cd70847e0abac2f0cd34797d0bbd2f25f37fe56a Mon Sep 17 00:00:00 2001 From: Sergey Prigogin Date: Mon, 29 Mar 2010 03:41:33 +0000 Subject: [PATCH] Minor performance improvement. --- .../dom/parser/cpp/CPPASTImplicitName.java | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTImplicitName.java b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTImplicitName.java index d55297f69b2..cdd63b69d3f 100644 --- a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTImplicitName.java +++ b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTImplicitName.java @@ -18,13 +18,13 @@ import org.eclipse.cdt.core.dom.ast.IASTImplicitName; import org.eclipse.cdt.core.dom.ast.IASTImplicitNameOwner; import org.eclipse.cdt.core.dom.ast.IASTNode; import org.eclipse.cdt.core.parser.IToken; +import org.eclipse.cdt.core.parser.Keywords; import org.eclipse.cdt.internal.core.dom.parser.ASTNode; public class CPPASTImplicitName extends CPPASTName implements IASTImplicitName { - - private boolean alternate = false; - private boolean isOperator = false; + private boolean alternate; + private boolean isOperator; public CPPASTImplicitName(char[] name, IASTNode parent) { super(name); @@ -52,7 +52,7 @@ public class CPPASTImplicitName extends CPPASTName implements IASTImplicitName { @Override public boolean accept(ASTVisitor action) { - if ((!alternate && action.shouldVisitImplicitNames) || + if ((!alternate && action.shouldVisitImplicitNames) || (alternate && action.shouldVisitImplicitNameAlternates)) { switch (action.visit(this)) { case ASTVisitor.PROCESS_ABORT: return false; @@ -80,11 +80,11 @@ public class CPPASTImplicitName extends CPPASTName implements IASTImplicitName { @Override public boolean isReference() { return true; - } + } /** * Utility method for setting offsets using operator syntax. - * + * * @param trailing true for trailing syntax, false for leading syntax */ public void computeOperatorOffsets(IASTNode relativeNode, boolean trailing) { @@ -95,14 +95,14 @@ public class CPPASTImplicitName extends CPPASTName implements IASTImplicitName { try { first = trailing ? relativeNode.getTrailingSyntax() : relativeNode.getLeadingSyntax(); - int offset = ((ASTNode)relativeNode).getOffset() + first.getOffset(); + int offset = ((ASTNode) relativeNode).getOffset() + first.getOffset(); if (trailing) - offset += ((ASTNode)relativeNode).getLength(); + offset += ((ASTNode) relativeNode).getLength(); OverloadableOperator oo = OverloadableOperator.valueOf(first); if ((first.getNext() == null && oo != null) || - Arrays.equals(first.getCharImage(), "delete".toCharArray()) || //$NON-NLS-1$ - Arrays.equals(first.getCharImage(), "new".toCharArray())) { //$NON-NLS-1$ + Arrays.equals(first.getCharImage(), Keywords.cDELETE) || + Arrays.equals(first.getCharImage(), Keywords.cNEW)) { int length = first.getLength(); setOffsetAndLength(offset, length); } else {