diff --git a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/c/CVisitor.java b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/c/CVisitor.java index 84563324521..5901eb7bc2c 100644 --- a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/c/CVisitor.java +++ b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/c/CVisitor.java @@ -118,14 +118,14 @@ public class CVisitor { } @Override public int visit(IASTName name) { - if ( name.getBinding() != null ) { + if (name.getBinding() != null) { try { IScope scope = name.resolveBinding().getScope(); - if ( scope != null ) + if (scope != null) ASTInternal.removeBinding(scope, name.resolveBinding()); - } catch ( DOMException e ) { + } catch (DOMException e) { } - name.setBinding( null ); + name.setBinding(null); } return PROCESS_CONTINUE; @@ -149,11 +149,10 @@ public class CVisitor { } private void addProblem(IASTProblem problem) { - if( problems.length == numFound ) // if the found array is full, then double the array - { - IASTProblem [] old = problems; - problems = new IASTProblem[ old.length * 2 ]; - for( int j = 0; j < old.length; ++j ) + if (problems.length == numFound) { // if the found array is full, then double the array + IASTProblem[] old = problems; + problems = new IASTProblem[old.length * 2]; + for (int j = 0; j < old.length; ++j) problems[j] = old[j]; } problems[numFound++] = problem; @@ -182,7 +181,7 @@ public class CVisitor { */ @Override public int visit(IASTDeclaration declaration) { - if ( declaration instanceof IASTProblemHolder ) + if (declaration instanceof IASTProblemHolder) addProblem(((IASTProblemHolder)declaration).getProblem()); return PROCESS_CONTINUE; @@ -193,7 +192,7 @@ public class CVisitor { */ @Override public int visit(IASTExpression expression) { - if ( expression instanceof IASTProblemHolder ) + if (expression instanceof IASTProblemHolder) addProblem(((IASTProblemHolder)expression).getProblem()); return PROCESS_CONTINUE; @@ -204,7 +203,7 @@ public class CVisitor { */ @Override public int visit(IASTStatement statement) { - if ( statement instanceof IASTProblemHolder ) + if (statement instanceof IASTProblemHolder) addProblem(((IASTProblemHolder)statement).getProblem()); return PROCESS_CONTINUE; @@ -215,7 +214,7 @@ public class CVisitor { */ @Override public int visit(IASTTypeId typeId) { - if ( typeId instanceof IASTProblemHolder ) + if (typeId instanceof IASTProblemHolder) addProblem(((IASTProblemHolder)typeId).getProblem()); return PROCESS_CONTINUE; @@ -237,11 +236,11 @@ public class CVisitor { boolean compositeTypeDeclared = false; private void addName(IASTName name) { - if( declsFound.length == numFound ) // if the found array is full, then double the array + if (declsFound.length == numFound) // if the found array is full, then double the array { - IASTName [] old = declsFound; - declsFound = new IASTName[ old.length * 2 ]; - for( int j = 0; j < old.length; ++j ) + IASTName[] old = declsFound; + declsFound = new IASTName[old.length * 2]; + for (int j = 0; j < old.length; ++j) declsFound[j] = old[j]; } declsFound[numFound++] = name; @@ -277,32 +276,32 @@ public class CVisitor { public int visit(IASTDeclarator declarator) { //GCC allows declarations in expressions, so we have to continue from the //declarator in case there is something in the initializer expression - if ( declarator == null || declarator.getName() == null || declarator.getName().toCharArray().length == 0 ) return PROCESS_CONTINUE; + if (declarator == null || declarator.getName() == null || declarator.getName().toCharArray().length == 0) return PROCESS_CONTINUE; //if the binding is something not declared in a declarator, continue - if( binding instanceof ICompositeType ) return PROCESS_CONTINUE; - if( binding instanceof IEnumeration ) return PROCESS_CONTINUE; + if (binding instanceof ICompositeType) return PROCESS_CONTINUE; + if (binding instanceof IEnumeration) return PROCESS_CONTINUE; IASTNode parent = declarator.getParent(); while (parent != null && !(parent instanceof IASTDeclaration || parent instanceof IASTParameterDeclaration)) parent = parent.getParent(); - if ( parent instanceof IASTDeclaration ) { - if ( parent instanceof IASTFunctionDefinition ) { - if ( declarator.getName() != null && declarator.getName().resolveBinding() == binding ) { + if (parent instanceof IASTDeclaration) { + if (parent instanceof IASTFunctionDefinition) { + if (declarator.getName() != null && declarator.getName().resolveBinding() == binding) { addName(declarator.getName()); } - } else if ( parent instanceof IASTSimpleDeclaration ) { + } else if (parent instanceof IASTSimpleDeclaration) { // prototype parameter with no identifier isn't a declaration of the K&R C parameter -// if ( binding instanceof CKnRParameter && declarator.getName().toCharArray().length == 0 ) +// if (binding instanceof CKnRParameter && declarator.getName().toCharArray().length == 0) // return PROCESS_CONTINUE; - if ( (declarator.getName() != null && declarator.getName().resolveBinding() == binding) ) { + if ((declarator.getName() != null && declarator.getName().resolveBinding() == binding)) { addName(declarator.getName()); } } - } else if ( parent instanceof IASTParameterDeclaration ) { - if ( declarator.getName() != null && declarator.getName().resolveBinding() == binding ) { + } else if (parent instanceof IASTParameterDeclaration) { + if (declarator.getName() != null && declarator.getName().resolveBinding() == binding) { addName(declarator.getName()); } } @@ -315,27 +314,27 @@ public class CVisitor { */ @Override public int visit(IASTDeclSpecifier declSpec) { - if ( compositeTypeDeclared && declSpec instanceof ICASTTypedefNameSpecifier ) + if (compositeTypeDeclared && declSpec instanceof ICASTTypedefNameSpecifier) return PROCESS_CONTINUE; //if the binding isn't declared in a decl spec, skip it - if( !(binding instanceof ICompositeType) && !(binding instanceof IEnumeration) ) + if (!(binding instanceof ICompositeType) && !(binding instanceof IEnumeration)) return PROCESS_CONTINUE; - if ( binding instanceof ICompositeType && declSpec instanceof IASTCompositeTypeSpecifier ) { - if( ((IASTCompositeTypeSpecifier)declSpec).getName().resolveBinding() == binding) { + if (binding instanceof ICompositeType && declSpec instanceof IASTCompositeTypeSpecifier) { + if (((IASTCompositeTypeSpecifier)declSpec).getName().resolveBinding() == binding) { compositeTypeDeclared = true; addName(((IASTCompositeTypeSpecifier)declSpec).getName()); } - } else if ( binding instanceof IEnumeration && declSpec instanceof IASTEnumerationSpecifier ) { - if( ((IASTEnumerationSpecifier)declSpec).getName().resolveBinding() == binding ) { + } else if (binding instanceof IEnumeration && declSpec instanceof IASTEnumerationSpecifier) { + if (((IASTEnumerationSpecifier)declSpec).getName().resolveBinding() == binding) { compositeTypeDeclared = true; addName(((IASTEnumerationSpecifier)declSpec).getName()); } - } else if( declSpec instanceof IASTElaboratedTypeSpecifier ) { - if( compositeTypeDeclared ){ + } else if (declSpec instanceof IASTElaboratedTypeSpecifier) { + if (compositeTypeDeclared) { IASTNode parent = declSpec.getParent(); - if( !(parent instanceof IASTSimpleDeclaration) || ((IASTSimpleDeclaration)parent).getDeclarators().length > 0 ){ + if (!(parent instanceof IASTSimpleDeclaration) || ((IASTSimpleDeclaration)parent).getDeclarators().length > 0) { return PROCESS_CONTINUE; } } @@ -353,8 +352,8 @@ public class CVisitor { */ @Override public int visit(IASTEnumerator enumerator) { - if( binding instanceof IEnumerator && enumerator.getName().resolveBinding() == binding ){ - addName( enumerator.getName() ); + if (binding instanceof IEnumerator && enumerator.getName().resolveBinding() == binding) { + addName(enumerator.getName()); } return PROCESS_CONTINUE; @@ -365,8 +364,8 @@ public class CVisitor { */ @Override public int visit(IASTStatement statement) { - if ( statement instanceof IASTLabelStatement && binding instanceof ILabel ){ - if ( ((IASTLabelStatement)statement).getName().resolveBinding() == binding ) + if (statement instanceof IASTLabelStatement && binding instanceof ILabel) { + if (((IASTLabelStatement)statement).getName().resolveBinding() == binding) addName(((IASTLabelStatement)statement).getName()); return PROCESS_SKIP; } @@ -377,7 +376,7 @@ public class CVisitor { public static class CollectReferencesAction extends CASTVisitor { private static final int DEFAULT_LIST_SIZE = 8; - private IASTName [] refs; + private IASTName[] refs; private IBinding binding; private int idx = 0; private int kind; @@ -387,16 +386,16 @@ public class CVisitor { private static final int KIND_TYPE = 3; - public CollectReferencesAction( IBinding binding ){ + public CollectReferencesAction(IBinding binding) { this.binding = binding; - this.refs = new IASTName[ DEFAULT_LIST_SIZE ]; + this.refs = new IASTName[DEFAULT_LIST_SIZE]; shouldVisitNames = true; - if( binding instanceof ILabel ) + if (binding instanceof ILabel) kind = KIND_LABEL; - else if( binding instanceof ICompositeType || + else if (binding instanceof ICompositeType || binding instanceof ITypedef || - binding instanceof IEnumeration ) + binding instanceof IEnumeration) { kind = KIND_TYPE; } else @@ -404,40 +403,40 @@ public class CVisitor { } @Override - public int visit( IASTName name ){ + public int visit(IASTName name) { ASTNodeProperty prop = name.getPropertyInParent(); - switch( kind ){ + switch(kind) { case KIND_LABEL: - if( prop == IASTGotoStatement.NAME ) + if (prop == IASTGotoStatement.NAME) break; return PROCESS_CONTINUE; case KIND_TYPE: - if( prop == IASTNamedTypeSpecifier.NAME ) + if (prop == IASTNamedTypeSpecifier.NAME) break; - else if( prop == IASTElaboratedTypeSpecifier.TYPE_NAME ){ + else if (prop == IASTElaboratedTypeSpecifier.TYPE_NAME) { IASTNode p = name.getParent().getParent(); - if( !(p instanceof IASTSimpleDeclaration) || - ((IASTSimpleDeclaration)p).getDeclarators().length > 0 ) + if (!(p instanceof IASTSimpleDeclaration) || + ((IASTSimpleDeclaration)p).getDeclarators().length > 0) { break; } } return PROCESS_CONTINUE; case KIND_OBJ_FN: - if( prop == IASTIdExpression.ID_NAME || + if (prop == IASTIdExpression.ID_NAME || prop == IASTFieldReference.FIELD_NAME || - prop == ICASTFieldDesignator.FIELD_NAME ) + prop == ICASTFieldDesignator.FIELD_NAME) { break; } return PROCESS_CONTINUE; } - if( CharArrayUtils.equals(name.toCharArray(), binding.getNameCharArray()) ) - if (sameBinding(name.resolveBinding(), binding)){ - if( refs.length == idx ){ - IASTName [] temp = new IASTName[ refs.length * 2 ]; - System.arraycopy( refs, 0, temp, 0, refs.length ); + if (CharArrayUtils.equals(name.toCharArray(), binding.getNameCharArray())) + if (sameBinding(name.resolveBinding(), binding)) { + if (refs.length == idx) { + IASTName[] temp = new IASTName[refs.length * 2]; + System.arraycopy(refs, 0, temp, 0, refs.length); refs = temp; } refs[idx++] = name; @@ -453,10 +452,10 @@ public class CVisitor { return false; } - public IASTName[] getReferences(){ - if( idx < refs.length ){ - IASTName [] temp = new IASTName[ idx ]; - System.arraycopy( refs, 0, temp, 0, idx ); + public IASTName[] getReferences() { + if (idx < refs.length) { + IASTName[] temp = new IASTName[idx]; + System.arraycopy(refs, 0, temp, 0, idx); refs = temp; } return refs; @@ -468,7 +467,7 @@ public class CVisitor { private static final String SIZE_T = "size_t"; //$NON-NLS-1$ private static final String PTRDIFF_T = "ptrdiff_t"; //$NON-NLS-1$ public static final String EMPTY_STRING = ""; //$NON-NLS-1$ - public static final char [] EMPTY_CHAR_ARRAY = "".toCharArray(); //$NON-NLS-1$ + public static final char[] EMPTY_CHAR_ARRAY = "".toCharArray(); //$NON-NLS-1$ //lookup bits private static final int COMPLETE = 0; private static final int CURRENT_SCOPE = 1; @@ -480,132 +479,132 @@ public class CVisitor { protected static final int AT_BEGINNING = 1; protected static final int AT_NEXT = 2; - static protected void createBinding( IASTName name ){ + static protected void createBinding(IASTName name) { IBinding binding = null; IASTNode parent = name.getParent(); - if( parent instanceof CASTIdExpression ){ - binding = resolveBinding( parent, COMPLETE | INCLUDE_BLOCK_ITEM ); - } else if( parent instanceof ICASTTypedefNameSpecifier ){ - binding = resolveBinding( parent ); - } else if( parent instanceof IASTFieldReference ){ - binding = (IBinding) findBinding( (IASTFieldReference) parent, false ); + if (parent instanceof CASTIdExpression) { + binding = resolveBinding(parent, COMPLETE | INCLUDE_BLOCK_ITEM); + } else if (parent instanceof ICASTTypedefNameSpecifier) { + binding = resolveBinding(parent); + } else if (parent instanceof IASTFieldReference) { + binding = (IBinding) findBinding((IASTFieldReference) parent, false); if (binding == null) { binding = new ProblemBinding(name, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray()); } - } else if( parent instanceof IASTDeclarator ){ - binding = createBinding( (IASTDeclarator) parent, name ); - } else if( parent instanceof ICASTCompositeTypeSpecifier ){ - binding = createBinding( (ICASTCompositeTypeSpecifier) parent ); - } else if( parent instanceof ICASTElaboratedTypeSpecifier ){ - binding = createBinding( (ICASTElaboratedTypeSpecifier) parent ); - } else if( parent instanceof IASTStatement ){ - binding = createBinding ( (IASTStatement) parent ); - } else if( parent instanceof ICASTEnumerationSpecifier ){ - binding = createBinding( (ICASTEnumerationSpecifier) parent ); - } else if( parent instanceof IASTEnumerator ) { - binding = createBinding( (IASTEnumerator) parent ); - } else if( parent instanceof ICASTFieldDesignator ) { - binding = resolveBinding( parent ); + } else if (parent instanceof IASTDeclarator) { + binding = createBinding((IASTDeclarator) parent, name); + } else if (parent instanceof ICASTCompositeTypeSpecifier) { + binding = createBinding((ICASTCompositeTypeSpecifier) parent); + } else if (parent instanceof ICASTElaboratedTypeSpecifier) { + binding = createBinding((ICASTElaboratedTypeSpecifier) parent); + } else if (parent instanceof IASTStatement) { + binding = createBinding ((IASTStatement) parent); + } else if (parent instanceof ICASTEnumerationSpecifier) { + binding = createBinding((ICASTEnumerationSpecifier) parent); + } else if (parent instanceof IASTEnumerator) { + binding = createBinding((IASTEnumerator) parent); + } else if (parent instanceof ICASTFieldDesignator) { + binding = resolveBinding(parent); } - name.setBinding( binding ); + name.setBinding(binding); } - private static IBinding createBinding( ICASTEnumerationSpecifier enumeration ){ + private static IBinding createBinding(ICASTEnumerationSpecifier enumeration) { IASTName name = enumeration.getName(); - IScope scope = getContainingScope( enumeration ); + IScope scope = getContainingScope(enumeration); IBinding binding= null; if (scope != null) { try { - binding = scope.getBinding( name, false ); - } catch ( DOMException e ) { + binding = scope.getBinding(name, false); + } catch (DOMException e) { } } if (binding != null && !(binding instanceof IIndexBinding)) { if (binding instanceof IEnumeration) { if (binding instanceof CEnumeration) { - ((CEnumeration)binding).addDefinition( name ); + ((CEnumeration)binding).addDefinition(name); } } else { - return new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray() ); + return new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray()); } } else { - binding = new CEnumeration( name ); + binding = new CEnumeration(name); try { ASTInternal.addName(scope, name); - } catch ( DOMException e1 ) { + } catch (DOMException e1) { } } return binding; } - private static IBinding createBinding( IASTEnumerator enumerator ){ - IEnumerator binding = new CEnumerator( enumerator ); + private static IBinding createBinding(IASTEnumerator enumerator) { + IEnumerator binding = new CEnumerator(enumerator); try { - ASTInternal.addName(binding.getScope(), enumerator.getName() ); - } catch ( DOMException e ) { + ASTInternal.addName(binding.getScope(), enumerator.getName()); + } catch (DOMException e) { } return binding; } - private static IBinding createBinding( IASTStatement statement ){ - if( statement instanceof IASTGotoStatement ){ - char [] gotoName = ((IASTGotoStatement)statement).getName().toCharArray(); - IScope scope = getContainingScope( statement ); - if( scope != null && scope instanceof ICFunctionScope ){ + private static IBinding createBinding(IASTStatement statement) { + if (statement instanceof IASTGotoStatement) { + char[] gotoName = ((IASTGotoStatement)statement).getName().toCharArray(); + IScope scope = getContainingScope(statement); + if (scope != null && scope instanceof ICFunctionScope) { CFunctionScope functionScope = (CFunctionScope) scope; - ILabel [] labels = functionScope.getLabels(); + ILabel[] labels = functionScope.getLabels(); for (ILabel label : labels) { - if( CharArrayUtils.equals( label.getNameCharArray(), gotoName) ){ + if (CharArrayUtils.equals(label.getNameCharArray(), gotoName)) { return label; } } //label not found - return new CLabel.CLabelProblem( ((IASTGotoStatement)statement).getName(), IProblemBinding.SEMANTIC_LABEL_STATEMENT_NOT_FOUND, gotoName ); + return new CLabel.CLabelProblem(((IASTGotoStatement)statement).getName(), IProblemBinding.SEMANTIC_LABEL_STATEMENT_NOT_FOUND, gotoName); } - } else if( statement instanceof IASTLabelStatement ){ + } else if (statement instanceof IASTLabelStatement) { IASTName name = ((IASTLabelStatement)statement).getName(); - IBinding binding = new CLabel( name ); + IBinding binding = new CLabel(name); try { IScope scope = binding.getScope(); if (scope instanceof ICFunctionScope) - ASTInternal.addName(binding.getScope(), name ); - } catch ( DOMException e ) { + ASTInternal.addName(binding.getScope(), name); + } catch (DOMException e) { } return binding; } return null; } - private static IBinding createBinding( ICASTElaboratedTypeSpecifier elabTypeSpec ){ + private static IBinding createBinding(ICASTElaboratedTypeSpecifier elabTypeSpec) { IASTNode parent = elabTypeSpec.getParent(); - if( parent instanceof IASTDeclaration ){ + if (parent instanceof IASTDeclaration) { int bits = TAGS; - if( parent instanceof IASTSimpleDeclaration && ((IASTSimpleDeclaration)parent).getDeclarators().length == 0 ){ + if (parent instanceof IASTSimpleDeclaration && ((IASTSimpleDeclaration)parent).getDeclarators().length == 0) { bits |= CURRENT_SCOPE; } IASTName name = elabTypeSpec.getName(); - IBinding binding = resolveBinding( elabTypeSpec, bits ); - if( binding != null ){ - if( binding instanceof CEnumeration ){ - ((CEnumeration)binding).addDeclaration( name ); + IBinding binding = resolveBinding(elabTypeSpec, bits); + if (binding != null) { + if (binding instanceof CEnumeration) { + ((CEnumeration)binding).addDeclaration(name); } } else { - if( elabTypeSpec.getKind() == IASTElaboratedTypeSpecifier.k_enum ){ - binding = new CEnumeration( name ); + if (elabTypeSpec.getKind() == IASTElaboratedTypeSpecifier.k_enum) { + binding = new CEnumeration(name); } else { - binding = new CStructure( name ); + binding = new CStructure(name); } try { - ASTInternal.addName(binding.getScope(), name ); - } catch ( DOMException e ) { + ASTInternal.addName(binding.getScope(), name); + } catch (DOMException e) { } } return binding; - } else if( parent instanceof IASTTypeId || parent instanceof IASTParameterDeclaration ){ - IASTNode blockItem = getContainingBlockItem( parent ); + } else if (parent instanceof IASTTypeId || parent instanceof IASTParameterDeclaration) { + IASTNode blockItem = getContainingBlockItem(parent); try { - return (IBinding) findBinding( blockItem, elabTypeSpec.getName(), COMPLETE | TAGS ); - } catch ( DOMException e ) { + return (IBinding) findBinding(blockItem, elabTypeSpec.getName(), COMPLETE | TAGS); + } catch (DOMException e) { return null; } } @@ -619,97 +618,97 @@ public class CVisitor { * @param prefix * @return */ - private static Object findBinding( IASTFieldReference fieldReference, boolean prefix ){ + private static Object findBinding(IASTFieldReference fieldReference, boolean prefix) { IASTExpression fieldOwner = fieldReference.getFieldOwner(); IType type = null; - if( fieldOwner instanceof IASTArraySubscriptExpression ){ - type = getExpressionType( ((IASTArraySubscriptExpression) fieldOwner).getArrayExpression() ); + if (fieldOwner instanceof IASTArraySubscriptExpression) { + type = getExpressionType(((IASTArraySubscriptExpression) fieldOwner).getArrayExpression()); } else { - type = getExpressionType( fieldOwner ); + type = getExpressionType(fieldOwner); } - while( type != null && type instanceof ITypeContainer) { + while (type != null && type instanceof ITypeContainer) { try { type = ((ITypeContainer)type).getType(); - } catch ( DOMException e ) { + } catch (DOMException e) { return e.getProblem(); } } - if( type != null && type instanceof ICompositeType ){ - if( prefix ){ - IBinding [] result = null; + if (type != null && type instanceof ICompositeType) { + if (prefix) { + IBinding[] result = null; try { - char [] p = fieldReference.getFieldName().toCharArray(); - IField [] fields = ((ICompositeType) type).getFields(); + char[] p = fieldReference.getFieldName().toCharArray(); + IField[] fields = ((ICompositeType) type).getFields(); for (IField field : fields) { - if( CharArrayUtils.equals( field.getNameCharArray(), 0, p.length, p, true ) ){ - result = (IBinding[]) ArrayUtil.append( IBinding.class, result, field ); + if (CharArrayUtils.equals(field.getNameCharArray(), 0, p.length, p, true)) { + result = (IBinding[]) ArrayUtil.append(IBinding.class, result, field); } } - return ArrayUtil.trim( IBinding.class, result ); - } catch ( DOMException e ) { + return ArrayUtil.trim(IBinding.class, result); + } catch (DOMException e) { return new IBinding[] { e.getProblem() }; } } try { - return ((ICompositeType) type).findField( fieldReference.getFieldName().toString() ); - } catch ( DOMException e ) { + return ((ICompositeType) type).findField(fieldReference.getFieldName().toString()); + } catch (DOMException e) { return e.getProblem(); } } return null; } - public static IType getExpressionType( IASTExpression expression ) { + public static IType getExpressionType(IASTExpression expression) { try{ - if( expression instanceof IASTIdExpression ){ + if (expression instanceof IASTIdExpression) { IBinding binding = ((IASTIdExpression)expression).getName().resolveBinding(); - if( binding instanceof IVariable ){ + if (binding instanceof IVariable) { return ((IVariable)binding).getType(); } else if (binding instanceof IFunction) { return ((IFunction)binding).getType(); } - } else if( expression instanceof IASTCastExpression ){ + } else if (expression instanceof IASTCastExpression) { IASTTypeId id = ((IASTCastExpression)expression).getTypeId(); - return createType( id.getAbstractDeclarator() ); - } else if( expression instanceof IASTFieldReference ){ + return createType(id.getAbstractDeclarator()); + } else if (expression instanceof IASTFieldReference) { IBinding binding = ((IASTFieldReference)expression).getFieldName().resolveBinding(); - if( binding instanceof IVariable ){ + if (binding instanceof IVariable) { return ((IVariable)binding).getType(); } - } else if( expression instanceof IASTFunctionCallExpression ){ - IType type = getExpressionType( ((IASTFunctionCallExpression)expression).getFunctionNameExpression() ); - while( type instanceof ITypeContainer ) + } else if (expression instanceof IASTFunctionCallExpression) { + IType type = getExpressionType(((IASTFunctionCallExpression)expression).getFunctionNameExpression()); + while (type instanceof ITypeContainer) type = ((ITypeContainer)type).getType(); - if( type instanceof IFunctionType ) + if (type instanceof IFunctionType) return ((IFunctionType)type).getReturnType(); - } else if( expression instanceof IASTUnaryExpression ) { - IType type = getExpressionType(((IASTUnaryExpression)expression).getOperand() ); + } else if (expression instanceof IASTUnaryExpression) { + IType type = getExpressionType(((IASTUnaryExpression)expression).getOperand()); int op = ((IASTUnaryExpression)expression).getOperator(); - if( op == IASTUnaryExpression.op_star && (type instanceof IPointerType || type instanceof IArrayType) ){ + if (op == IASTUnaryExpression.op_star && (type instanceof IPointerType || type instanceof IArrayType)) { return ((ITypeContainer)type).getType(); - } else if( op == IASTUnaryExpression.op_amper ){ - return new CPointerType( type, 0 ); + } else if (op == IASTUnaryExpression.op_amper) { + return new CPointerType(type, 0); } return type; - } else if( expression instanceof IASTLiteralExpression ){ - switch( ((IASTLiteralExpression) expression).getKind() ){ + } else if (expression instanceof IASTLiteralExpression) { + switch(((IASTLiteralExpression) expression).getKind()) { case IASTLiteralExpression.lk_char_constant: - return new CBasicType( IBasicType.t_char, 0, expression ); + return new CBasicType(IBasicType.t_char, 0, expression); case IASTLiteralExpression.lk_float_constant: - return new CBasicType( IBasicType.t_float, 0, expression ); + return new CBasicType(IBasicType.t_float, 0, expression); case IASTLiteralExpression.lk_integer_constant: - return new CBasicType( IBasicType.t_int, 0, expression ); + return new CBasicType(IBasicType.t_int, 0, expression); case IASTLiteralExpression.lk_string_literal: - IType type = new CBasicType( IBasicType.t_char, 0, expression ); - type = new CQualifierType( type, true, false, false ); - return new CPointerType( type, 0 ); + IType type = new CBasicType(IBasicType.t_char, 0, expression); + type = new CQualifierType(type, true, false, false); + return new CPointerType(type, 0); } - } else if( expression instanceof IASTBinaryExpression ){ + } else if (expression instanceof IASTBinaryExpression) { IASTBinaryExpression binary = (IASTBinaryExpression) expression; int op = binary.getOperator(); - switch( op ){ + switch(op) { case IASTBinaryExpression.op_lessEqual: case IASTBinaryExpression.op_lessThan: case IASTBinaryExpression.op_greaterEqual: @@ -718,7 +717,7 @@ public class CVisitor { case IASTBinaryExpression.op_logicalOr: case IASTBinaryExpression.op_equals: case IASTBinaryExpression.op_notequals: - CBasicType basicType = new CBasicType( IBasicType.t_int, 0 ); + CBasicType basicType = new CBasicType(IBasicType.t_int, 0); basicType.setValue(expression); return basicType; case IASTBinaryExpression.op_plus: @@ -757,82 +756,82 @@ public class CVisitor { } break; } - return getExpressionType(binary.getOperand1() ); - } else if( expression instanceof IASTUnaryExpression ) { + return getExpressionType(binary.getOperand1()); + } else if (expression instanceof IASTUnaryExpression) { int op = ((IASTUnaryExpression)expression).getOperator(); - if( op == IASTUnaryExpression.op_sizeof ){ - IScope scope = getContainingScope( expression ); + if (op == IASTUnaryExpression.op_sizeof) { + IScope scope = getContainingScope(expression); if (scope != null) { - IBinding [] bs = scope.find( SIZE_T ); - if( bs.length > 0 && bs[0] instanceof IType ){ + IBinding[] bs = scope.find(SIZE_T); + if (bs.length > 0 && bs[0] instanceof IType) { return (IType) bs[0]; } } - return new CBasicType( IBasicType.t_int, CBasicType.IS_LONG | CBasicType.IS_UNSIGNED, expression ); + return new CBasicType(IBasicType.t_int, CBasicType.IS_LONG | CBasicType.IS_UNSIGNED, expression); } - IType type = getExpressionType(((IASTUnaryExpression)expression).getOperand() ); + IType type = getExpressionType(((IASTUnaryExpression)expression).getOperand()); - if( op == IASTUnaryExpression.op_star && (type instanceof IPointerType || type instanceof IArrayType) ){ + if (op == IASTUnaryExpression.op_star && (type instanceof IPointerType || type instanceof IArrayType)) { try { return ((ITypeContainer)type).getType(); } catch (DOMException e) { return e.getProblem(); } - } else if( op == IASTUnaryExpression.op_amper ){ - return new CPointerType( type, 0 ); - } else if ( type instanceof CBasicType ){ - ((CBasicType)type).setValue( expression ); + } else if (op == IASTUnaryExpression.op_amper) { + return new CPointerType(type, 0); + } else if (type instanceof CBasicType) { + ((CBasicType)type).setValue(expression); } return type; - } else if( expression instanceof IASTFieldReference ){ - IBinding binding = (IBinding) findBinding( (IASTFieldReference) expression, false ); - if( binding instanceof IVariable ) + } else if (expression instanceof IASTFieldReference) { + IBinding binding = (IBinding) findBinding((IASTFieldReference) expression, false); + if (binding instanceof IVariable) return ((IVariable)binding).getType(); - else if( binding instanceof IFunction ) + else if (binding instanceof IFunction) return ((IFunction)binding).getType(); - else if( binding instanceof IEnumerator ) + else if (binding instanceof IEnumerator) return ((IEnumerator)binding).getType(); - } else if( expression instanceof IASTExpressionList ){ - IASTExpression [] exps = ((IASTExpressionList)expression).getExpressions(); - return getExpressionType( exps[ exps.length - 1 ] ); - } else if( expression instanceof IASTTypeIdExpression ){ + } else if (expression instanceof IASTExpressionList) { + IASTExpression[] exps = ((IASTExpressionList)expression).getExpressions(); + return getExpressionType(exps[exps.length - 1]); + } else if (expression instanceof IASTTypeIdExpression) { IASTTypeIdExpression typeidExp = (IASTTypeIdExpression) expression; - if( typeidExp.getOperator() == IASTTypeIdExpression.op_sizeof ){ - IScope scope = getContainingScope( typeidExp ); - IBinding [] bs = scope.find( SIZE_T ); - if( bs.length > 0 && bs[0] instanceof IType ){ + if (typeidExp.getOperator() == IASTTypeIdExpression.op_sizeof) { + IScope scope = getContainingScope(typeidExp); + IBinding[] bs = scope.find(SIZE_T); + if (bs.length > 0 && bs[0] instanceof IType) { return (IType) bs[0]; } - return new CBasicType( IBasicType.t_int, CBasicType.IS_LONG | CBasicType.IS_UNSIGNED ); + return new CBasicType(IBasicType.t_int, CBasicType.IS_LONG | CBasicType.IS_UNSIGNED); } - return createType( typeidExp.getTypeId().getAbstractDeclarator() ); - } else if( expression instanceof IASTArraySubscriptExpression ){ - IType t = getExpressionType( ((IASTArraySubscriptExpression) expression).getArrayExpression() ); - if( t instanceof IPointerType ) + return createType(typeidExp.getTypeId().getAbstractDeclarator()); + } else if (expression instanceof IASTArraySubscriptExpression) { + IType t = getExpressionType(((IASTArraySubscriptExpression) expression).getArrayExpression()); + if (t instanceof IPointerType) return ((IPointerType)t).getType(); - else if( t instanceof IArrayType ) + else if (t instanceof IArrayType) return ((IArrayType)t).getType(); - } else if( expression instanceof IGNUASTCompoundStatementExpression ){ + } else if (expression instanceof IGNUASTCompoundStatementExpression) { IASTCompoundStatement compound = ((IGNUASTCompoundStatementExpression)expression).getCompoundStatement(); - IASTStatement [] statements = compound.getStatements(); - if( statements.length > 0 ){ - IASTStatement st = statements[ statements.length - 1 ]; - if( st instanceof IASTExpressionStatement ) - return getExpressionType( ((IASTExpressionStatement)st).getExpression() ); + IASTStatement[] statements = compound.getStatements(); + if (statements.length > 0) { + IASTStatement st = statements[statements.length - 1]; + if (st instanceof IASTExpressionStatement) + return getExpressionType(((IASTExpressionStatement)st).getExpression()); } - } else if( expression instanceof IASTConditionalExpression ){ + } else if (expression instanceof IASTConditionalExpression) { final IASTConditionalExpression conditional = (IASTConditionalExpression) expression; IASTExpression positiveExpression = conditional.getPositiveResultExpression(); if (positiveExpression == null) { positiveExpression= conditional.getLogicalConditionExpression(); } IType t2 = getExpressionType(positiveExpression); - IType t3 = getExpressionType( conditional.getNegativeResultExpression() ); - if( t3 instanceof IPointerType || t2 == null ) + IType t3 = getExpressionType(conditional.getNegativeResultExpression()); + if (t3 instanceof IPointerType || t2 == null) return t3; return t2; } - } catch( DOMException e ){ + } catch(DOMException e) { return e.getProblem(); } return null; @@ -851,47 +850,47 @@ public class CVisitor { */ private static IBinding createBinding(IASTDeclarator declarator, IASTName name) { IBinding binding = null; - if( declarator instanceof ICASTKnRFunctionDeclarator ){ + if (declarator instanceof ICASTKnRFunctionDeclarator) { IASTNode parent = declarator.getParent(); - if ( CharArrayUtils.equals(declarator.getName().toCharArray(), name.toCharArray()) ){ - binding = resolveBinding( parent, CURRENT_SCOPE ); - if( binding != null && binding instanceof IIndexBinding == false) { - if( binding instanceof ICInternalFunction ) - ((ICInternalFunction)binding).addDeclarator( (ICASTKnRFunctionDeclarator) declarator ); + if (CharArrayUtils.equals(declarator.getName().toCharArray(), name.toCharArray())) { + binding = resolveBinding(parent, CURRENT_SCOPE); + if (binding != null && binding instanceof IIndexBinding == false) { + if (binding instanceof ICInternalFunction) + ((ICInternalFunction)binding).addDeclarator((ICASTKnRFunctionDeclarator) declarator); else - binding = new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray() ); + binding = new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray()); } else { binding = createBinding(declarator); } } else { // createBinding for one of the ICASTKnRFunctionDeclarator's parameterNames IBinding f = declarator.getName().resolveBinding(); - if( f instanceof CFunction ){ - binding = ((CFunction) f).resolveParameter( name ); + if (f instanceof CFunction) { + binding = ((CFunction) f).resolveParameter(name); } - if ( declarator.getParent() instanceof IASTFunctionDefinition ) { + if (declarator.getParent() instanceof IASTFunctionDefinition) { IScope scope = ((IASTCompoundStatement)((IASTFunctionDefinition)declarator.getParent()).getBody()).getScope(); - if ( scope != null && binding != null ) + if (scope != null && binding != null) try { - ASTInternal.addName( scope, name); - } catch ( DOMException e ) { + ASTInternal.addName(scope, name); + } catch (DOMException e) { } } } } else { - binding = createBinding( declarator ); + binding = createBinding(declarator); } return binding; } - private static IBinding createBinding( IASTDeclarator declarator ){ + private static IBinding createBinding(IASTDeclarator declarator) { IASTNode parent = declarator.getParent(); - while( parent instanceof IASTDeclarator ){ + while (parent instanceof IASTDeclarator) { parent = parent.getParent(); } - while( declarator.getNestedDeclarator() != null ) + while (declarator.getNestedDeclarator() != null) declarator = declarator.getNestedDeclarator(); IASTFunctionDeclarator funcDeclarator= null; @@ -907,15 +906,16 @@ public class CVisitor { } node= node.getParent(); } - while (node instanceof IASTDeclarator); + while (node instanceof IASTDeclarator) + ; - IScope scope = getContainingScope( parent ); + IScope scope = getContainingScope(parent); ASTNodeProperty prop = parent.getPropertyInParent(); - if( prop == IASTDeclarationStatement.DECLARATION ){ + if (prop == IASTDeclarationStatement.DECLARATION) { //implicit scope, see 6.8.4-3 prop = parent.getParent().getPropertyInParent(); - if( prop != IASTCompoundStatement.NESTED_STATEMENT ) + if (prop != IASTCompoundStatement.NESTED_STATEMENT) scope = null; } @@ -923,11 +923,11 @@ public class CVisitor { IBinding binding = null; try { - binding = ( scope != null ) ? scope.getBinding( name, false ) : null; - } catch ( DOMException e1 ) { + binding = (scope != null) ? scope.getBinding(name, false) : null; + } catch (DOMException e1) { } - if( parent instanceof IASTParameterDeclaration || parent.getPropertyInParent() == ICASTKnRFunctionDeclarator.FUNCTION_PARAMETER ){ + if (parent instanceof IASTParameterDeclaration || parent.getPropertyInParent() == ICASTKnRFunctionDeclarator.FUNCTION_PARAMETER) { IASTDeclarator fdtor = (IASTDeclarator) parent.getParent(); IASTDeclarator nested= fdtor.getNestedDeclarator(); while (nested != null && nested.getPointerOperators().length == 0) { @@ -935,217 +935,218 @@ public class CVisitor { nested= nested.getNestedDeclarator(); } IBinding temp = fdtor.getName().resolveBinding(); - if( temp != null && temp instanceof CFunction ){ - binding = ((CFunction) temp).resolveParameter( name ); - } else if( temp instanceof IFunction ){ + if (temp != null && temp instanceof CFunction) { + binding = ((CFunction) temp).resolveParameter(name); + } else if (temp instanceof IFunction) { //problems with the function, still create binding for the parameter - binding = new CParameter( name ); + binding = new CParameter(name); } try { - if( scope != null && ASTInternal.getPhysicalNodeOfScope(scope) instanceof IASTTranslationUnit ){ + if (scope != null && ASTInternal.getPhysicalNodeOfScope(scope) instanceof IASTTranslationUnit) { return binding; } } catch (DOMException e) { } - } else if( funcDeclarator != null ){ - if( binding != null && !(binding instanceof IIndexBinding)) { - if( binding instanceof IFunction ){ + } else if (funcDeclarator != null) { + if (binding != null && !(binding instanceof IIndexBinding)) { + if (binding instanceof IFunction) { IFunction function = (IFunction) binding; - if( function instanceof CFunction ) { - ((CFunction)function).addDeclarator( funcDeclarator ); + if (function instanceof CFunction) { + ((CFunction)function).addDeclarator(funcDeclarator); } return function; } - binding = new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray() ); - } else if( parent instanceof IASTSimpleDeclaration && ((IASTSimpleDeclaration) parent).getDeclSpecifier().getStorageClass() == IASTDeclSpecifier.sc_typedef) - binding = new CTypedef( name ); + binding = new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_OVERLOAD, name.toCharArray()); + } else if (parent instanceof IASTSimpleDeclaration && ((IASTSimpleDeclaration) parent).getDeclSpecifier().getStorageClass() == IASTDeclSpecifier.sc_typedef) + binding = new CTypedef(name); else - binding = new CFunction( funcDeclarator ); - } else if( parent instanceof IASTSimpleDeclaration ){ + binding = new CFunction(funcDeclarator); + } else if (parent instanceof IASTSimpleDeclaration) { IASTSimpleDeclaration simpleDecl = (IASTSimpleDeclaration) parent; - if( simpleDecl.getDeclSpecifier().getStorageClass() == IASTDeclSpecifier.sc_typedef ){ - binding = new CTypedef( name ); + if (simpleDecl.getDeclSpecifier().getStorageClass() == IASTDeclSpecifier.sc_typedef) { + binding = new CTypedef(name); } else { IType t1 = null, t2 = null; - if( binding != null && !(binding instanceof IIndexBinding)) { - if( binding instanceof IParameter ){ - return new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_REDECLARATION, name.toCharArray() ); - } else if( binding instanceof IVariable ){ - t1 = createType( declarator ); + if (binding != null && !(binding instanceof IIndexBinding)) { + if (binding instanceof IParameter) { + return new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_REDECLARATION, name.toCharArray()); + } else if (binding instanceof IVariable) { + t1 = createType(declarator); try { t2 = ((IVariable)binding).getType(); - } catch ( DOMException e1 ) { + } catch (DOMException e1) { } - if( t1 != null && t2 != null && t1.isSameType( t2 ) ){ - if( binding instanceof CVariable ) - ((CVariable)binding).addDeclaration( name ); + if (t1 != null && t2 != null && t1.isSameType(t2)) { + if (binding instanceof CVariable) + ((CVariable)binding).addDeclaration(name); } else { - return new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_REDECLARATION, name.toCharArray() ); + return new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_REDECLARATION, name.toCharArray()); } } - } else if( simpleDecl.getParent() instanceof ICASTCompositeTypeSpecifier ){ - binding = new CField( name ); + } else if (simpleDecl.getParent() instanceof ICASTCompositeTypeSpecifier) { + binding = new CField(name); } else { - binding = new CVariable( name ); + binding = new CVariable(name); } } } - if( scope != null && binding != null ) + if (scope != null && binding != null) try { - ASTInternal.addName( scope, name ); - } catch ( DOMException e ) { + ASTInternal.addName(scope, name); + } catch (DOMException e) { } return binding; } - private static IBinding createBinding( ICASTCompositeTypeSpecifier compositeTypeSpec ){ + private static IBinding createBinding(ICASTCompositeTypeSpecifier compositeTypeSpec) { IScope scope = null; IBinding binding = null; IASTName name = compositeTypeSpec.getName(); try { - scope = getContainingScope( compositeTypeSpec ); - while( scope instanceof ICCompositeTypeScope ) + scope = getContainingScope(compositeTypeSpec); + while (scope instanceof ICCompositeTypeScope) scope = scope.getParent(); if (scope != null) { - binding = scope.getBinding( name, false ); - if( binding != null && !(binding instanceof IIndexBinding)){ + binding = scope.getBinding(name, false); + if (binding != null && !(binding instanceof IIndexBinding)) { if (binding instanceof CStructure) - ((CStructure)binding).addDefinition( compositeTypeSpec ); + ((CStructure)binding).addDefinition(compositeTypeSpec); return binding; } } } catch (DOMException e2) { } - binding = new CStructure( name ); + binding = new CStructure(name); try { scope= binding.getScope(); ASTInternal.addName(scope, name); - } catch ( DOMException e ) { + } catch (DOMException e) { } return binding; } - protected static IBinding resolveBinding( IASTNode node ){ - return resolveBinding( node, COMPLETE ); + protected static IBinding resolveBinding(IASTNode node) { + return resolveBinding(node, COMPLETE); } - protected static IBinding resolveBinding( IASTNode node, int bits ){ - if( node instanceof IASTFunctionDefinition ){ + + protected static IBinding resolveBinding(IASTNode node, int bits) { + if (node instanceof IASTFunctionDefinition) { IASTFunctionDefinition functionDef = (IASTFunctionDefinition) node; IASTFunctionDeclarator functionDeclartor = functionDef.getDeclarator(); IASTName name = functionDeclartor.getName(); - IASTNode blockItem = getContainingBlockItem( node ); + IASTNode blockItem = getContainingBlockItem(node); try { - return (IBinding) findBinding( blockItem, name, bits ); - } catch ( DOMException e ) { + return (IBinding) findBinding(blockItem, name, bits); + } catch (DOMException e) { return null; } - } else if( node instanceof IASTIdExpression ){ - IASTNode blockItem = getContainingBlockItem( node ); + } else if (node instanceof IASTIdExpression) { + IASTNode blockItem = getContainingBlockItem(node); try { - IBinding binding = (IBinding) findBinding( blockItem, ((IASTIdExpression)node).getName(), bits ); - if( binding instanceof IType && !(binding instanceof IProblemBinding) ){ - return new ProblemBinding( node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray() ); + IBinding binding = (IBinding) findBinding(blockItem, ((IASTIdExpression)node).getName(), bits); + if (binding instanceof IType && !(binding instanceof IProblemBinding) ) { + return new ProblemBinding(node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray()); } return binding; - } catch ( DOMException e ) { + } catch (DOMException e) { return null; } - } else if( node instanceof ICASTTypedefNameSpecifier ){ - IASTNode blockItem = getContainingBlockItem( node ); + } else if (node instanceof ICASTTypedefNameSpecifier) { + IASTNode blockItem = getContainingBlockItem(node); try { IASTName name= ((ICASTTypedefNameSpecifier)node).getName(); - IBinding binding = (IBinding) findBinding( blockItem, name, bits ); - if( binding == null ) - return new ProblemBinding( node, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray()); - if( binding instanceof IType ) + IBinding binding = (IBinding) findBinding(blockItem, name, bits); + if (binding == null) + return new ProblemBinding(node, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray()); + if (binding instanceof IType) return binding; - return new ProblemBinding( node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray() ); - } catch ( DOMException e ) { + return new ProblemBinding(node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray()); + } catch (DOMException e) { return null; } - } else if( node instanceof ICASTElaboratedTypeSpecifier ){ - IASTNode blockItem = getContainingBlockItem( node ); + } else if (node instanceof ICASTElaboratedTypeSpecifier) { + IASTNode blockItem = getContainingBlockItem(node); try { - return (IBinding) findBinding( blockItem, ((ICASTElaboratedTypeSpecifier)node).getName(), bits ); - } catch ( DOMException e ) { + return (IBinding) findBinding(blockItem, ((ICASTElaboratedTypeSpecifier)node).getName(), bits); + } catch (DOMException e) { return null; } - } else if( node instanceof ICASTCompositeTypeSpecifier ){ - IASTNode blockItem = getContainingBlockItem( node ); + } else if (node instanceof ICASTCompositeTypeSpecifier) { + IASTNode blockItem = getContainingBlockItem(node); try { - return (IBinding) findBinding( blockItem, ((ICASTCompositeTypeSpecifier)node).getName(), bits ); - } catch ( DOMException e ) { + return (IBinding) findBinding(blockItem, ((ICASTCompositeTypeSpecifier)node).getName(), bits); + } catch (DOMException e) { return null; } - } else if( node instanceof IASTTypeId ){ + } else if (node instanceof IASTTypeId) { IASTTypeId typeId = (IASTTypeId) node; IASTDeclSpecifier declSpec = typeId.getDeclSpecifier(); IASTName name = null; - if( declSpec instanceof ICASTElaboratedTypeSpecifier ){ + if (declSpec instanceof ICASTElaboratedTypeSpecifier) { name = ((ICASTElaboratedTypeSpecifier)declSpec).getName(); - } else if( declSpec instanceof ICASTCompositeTypeSpecifier ){ + } else if (declSpec instanceof ICASTCompositeTypeSpecifier) { name = ((ICASTCompositeTypeSpecifier)declSpec).getName(); - } else if( declSpec instanceof ICASTTypedefNameSpecifier ){ + } else if (declSpec instanceof ICASTTypedefNameSpecifier) { name = ((ICASTTypedefNameSpecifier)declSpec).getName(); } - if( name != null ){ + if (name != null) { IBinding binding = name.resolveBinding(); - if( binding instanceof IType ) + if (binding instanceof IType) return binding; - else if( binding != null ) - return new ProblemBinding( node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray() ); + else if (binding != null) + return new ProblemBinding(node, IProblemBinding.SEMANTIC_INVALID_TYPE, binding.getNameCharArray()); return null; } - } else if( node instanceof ICASTFieldDesignator ) { - IASTNode blockItem = getContainingBlockItem( node ); + } else if (node instanceof ICASTFieldDesignator) { + IASTNode blockItem = getContainingBlockItem(node); - if ( (blockItem instanceof IASTSimpleDeclaration || - (blockItem instanceof IASTDeclarationStatement && ((IASTDeclarationStatement)blockItem).getDeclaration() instanceof IASTSimpleDeclaration)) ) { + if ((blockItem instanceof IASTSimpleDeclaration || + (blockItem instanceof IASTDeclarationStatement && ((IASTDeclarationStatement)blockItem).getDeclaration() instanceof IASTSimpleDeclaration))) { IASTSimpleDeclaration simpleDecl = null; if (blockItem instanceof IASTDeclarationStatement && - ((IASTDeclarationStatement)blockItem).getDeclaration() instanceof IASTSimpleDeclaration ) + ((IASTDeclarationStatement)blockItem).getDeclaration() instanceof IASTSimpleDeclaration) simpleDecl = (IASTSimpleDeclaration)((IASTDeclarationStatement)blockItem).getDeclaration(); - else if ( blockItem instanceof IASTSimpleDeclaration ) + else if (blockItem instanceof IASTSimpleDeclaration) simpleDecl = (IASTSimpleDeclaration)blockItem; - if ( simpleDecl != null ) { + if (simpleDecl != null) { IBinding struct = null; - if ( simpleDecl.getDeclSpecifier() instanceof IASTNamedTypeSpecifier ) + if (simpleDecl.getDeclSpecifier() instanceof IASTNamedTypeSpecifier) struct = ((IASTNamedTypeSpecifier)simpleDecl.getDeclSpecifier()).getName().resolveBinding(); - else if ( simpleDecl.getDeclSpecifier() instanceof IASTElaboratedTypeSpecifier ) + else if (simpleDecl.getDeclSpecifier() instanceof IASTElaboratedTypeSpecifier) struct = ((IASTElaboratedTypeSpecifier)simpleDecl.getDeclSpecifier()).getName().resolveBinding(); - else if ( simpleDecl.getDeclSpecifier() instanceof IASTCompositeTypeSpecifier ) + else if (simpleDecl.getDeclSpecifier() instanceof IASTCompositeTypeSpecifier) struct = ((IASTCompositeTypeSpecifier)simpleDecl.getDeclSpecifier()).getName().resolveBinding(); - if ( struct instanceof CStructure ) { + if (struct instanceof CStructure) { try { return ((CStructure)struct).findField(((ICASTFieldDesignator)node).getName().toString()); - } catch ( DOMException e ) { + } catch (DOMException e) { return e.getProblem(); } - } else if ( struct instanceof ITypeContainer ) { + } else if (struct instanceof ITypeContainer) { IType type; try { type = ((ITypeContainer)struct).getType(); - while ( type instanceof ITypeContainer && !(type instanceof CStructure) ) { + while (type instanceof ITypeContainer && !(type instanceof CStructure)) { type = ((ITypeContainer)type).getType(); } - } catch ( DOMException e ) { + } catch (DOMException e) { return e.getProblem(); } - if ( type instanceof CStructure ) + if (type instanceof CStructure) try { return ((CStructure)type).findField(((ICASTFieldDesignator)node).getName().toString()); - } catch ( DOMException e1 ) { + } catch (DOMException e1) { return e1.getProblem(); } } @@ -1158,38 +1159,38 @@ public class CVisitor { /** * May return null, e.g. for parameter names in function-prototypes. */ - public static IScope getContainingScope( IASTNode node ){ - if( node == null ) + public static IScope getContainingScope(IASTNode node) { + if (node == null) return null; - while( node != null ){ - if( node instanceof IASTDeclaration ){ + while (node != null) { + if (node instanceof IASTDeclaration) { IASTNode parent = node.getParent(); - if( parent instanceof IASTTranslationUnit ){ + if (parent instanceof IASTTranslationUnit) { return ((IASTTranslationUnit)parent).getScope(); - } else if( parent instanceof IASTDeclarationStatement ){ - return getContainingScope( (IASTStatement) parent ); - } else if( parent instanceof IASTForStatement ){ + } else if (parent instanceof IASTDeclarationStatement) { + return getContainingScope((IASTStatement) parent); + } else if (parent instanceof IASTForStatement) { return ((IASTForStatement)parent).getScope(); - } else if( parent instanceof IASTCompositeTypeSpecifier ){ + } else if (parent instanceof IASTCompositeTypeSpecifier) { return ((IASTCompositeTypeSpecifier)parent).getScope(); - } else if( parent instanceof ICASTKnRFunctionDeclarator ){ + } else if (parent instanceof ICASTKnRFunctionDeclarator) { parent = ((IASTDeclarator)parent).getParent(); - if ( parent instanceof IASTFunctionDefinition ) { + if (parent instanceof IASTFunctionDefinition) { return ((IASTCompoundStatement)((IASTFunctionDefinition)parent).getBody()).getScope(); } } - } else if( node instanceof IASTStatement ) - return getContainingScope( (IASTStatement) node ); - else if( node instanceof IASTParameterDeclaration ){ + } else if (node instanceof IASTStatement) + return getContainingScope((IASTStatement) node); + else if (node instanceof IASTParameterDeclaration) { IASTNode parent = node.getParent(); - if( parent instanceof IASTStandardFunctionDeclarator ){ + if (parent instanceof IASTStandardFunctionDeclarator) { parent = ((IASTDeclarator)parent).getParent(); - if ( parent instanceof IASTFunctionDefinition ) + if (parent instanceof IASTFunctionDefinition) return ((IASTCompoundStatement)((IASTFunctionDefinition)parent).getBody()).getScope(); return null; // parameter name in function declarations } } - else if( node instanceof IASTEnumerator ){ + else if (node instanceof IASTEnumerator) { //put the enumerators in the same scope as the enumeration node = node.getParent(); } @@ -1199,34 +1200,34 @@ public class CVisitor { return null; } - public static IScope getContainingScope( IASTStatement statement ){ + public static IScope getContainingScope(IASTStatement statement) { IASTNode parent = statement.getParent(); IScope scope = null; - if( parent instanceof IASTCompoundStatement ){ + if (parent instanceof IASTCompoundStatement) { IASTCompoundStatement compound = (IASTCompoundStatement) parent; scope = compound.getScope(); - } else if( parent instanceof IASTStatement ){ - scope = getContainingScope( (IASTStatement)parent ); - } else if( parent instanceof IASTFunctionDefinition ){ - IASTFunctionDeclarator fnDeclarator = ((IASTFunctionDefinition) parent ).getDeclarator(); + } else if (parent instanceof IASTStatement) { + scope = getContainingScope((IASTStatement)parent); + } else if (parent instanceof IASTFunctionDefinition) { + IASTFunctionDeclarator fnDeclarator = ((IASTFunctionDefinition) parent).getDeclarator(); IBinding function = fnDeclarator.getName().resolveBinding(); try { - if( function instanceof IFunction ){ + if (function instanceof IFunction) { scope = ((IFunction)function).getFunctionScope(); - } else if( function instanceof ProblemBinding ) { + } else if (function instanceof ProblemBinding) { return (IScope) function; } - } catch ( DOMException e ) { + } catch (DOMException e) { return e.getProblem(); } } - if( statement instanceof IASTGotoStatement || statement instanceof IASTLabelStatement ){ + if (statement instanceof IASTGotoStatement || statement instanceof IASTLabelStatement) { //labels have function scope - while( scope != null && !(scope instanceof ICFunctionScope) ){ + while (scope != null && !(scope instanceof ICFunctionScope)) { try { scope = scope.getParent(); - } catch ( DOMException e ) { + } catch (DOMException e) { scope = e.getProblem(); break; } @@ -1236,31 +1237,31 @@ public class CVisitor { return scope; } - private static IASTNode getContainingBlockItem( IASTNode node ){ + private static IASTNode getContainingBlockItem(IASTNode node) { IASTNode parent = node.getParent(); - if( parent instanceof IASTDeclaration ){ + if (parent instanceof IASTDeclaration) { IASTNode p = parent.getParent(); - if( p instanceof IASTDeclarationStatement ) + if (p instanceof IASTDeclarationStatement) return p; return parent; } //if parent is something that can contain a declaration - else if ( parent instanceof IASTCompoundStatement || + else if (parent instanceof IASTCompoundStatement || parent instanceof IASTTranslationUnit || parent instanceof IASTForStatement || - parent instanceof IASTFunctionDeclarator ) + parent instanceof IASTFunctionDeclarator) { return node; } - return getContainingBlockItem( parent ); + return getContainingBlockItem(parent); } /** - * if (bits & PREFIX_LOOKUP) then returns IBinding [] + * if (bits & PREFIX_LOOKUP) then returns IBinding[] * otherwise returns IBinding */ - protected static Object findBinding( IASTNode blockItem, IASTName name, int bits ) throws DOMException{ + protected static Object findBinding(IASTNode blockItem, IASTName name, int bits) throws DOMException{ IIndexFileSet fileSet= IIndexFileSet.EMPTY; if (blockItem != null) { final IASTTranslationUnit tu= blockItem.getTranslationUnit(); @@ -1272,32 +1273,31 @@ public class CVisitor { } } - boolean prefix = ( bits & PREFIX_LOOKUP ) != 0; + boolean prefix = (bits & PREFIX_LOOKUP) != 0; @SuppressWarnings("unchecked") - Object binding = prefix ? new ObjectSet( 2 ) : null; + Object binding = prefix ? new ObjectSet(2) : null; IIndexBinding foundIndexBinding= null; CharArrayObjectMap prefixMap = prefix ? new CharArrayObjectMap(2) : null; - while( blockItem != null ){ - + while (blockItem != null) { IASTNode parent = blockItem.getParent(); - IASTNode [] nodes = null; + IASTNode[] nodes = null; IScope scope = null; - if( parent instanceof IASTCompoundStatement ){ + if (parent instanceof IASTCompoundStatement) { IASTCompoundStatement compound = (IASTCompoundStatement) parent; scope = compound.getScope(); - if( parent.getParent() instanceof IASTFunctionDefinition ){ + if (parent.getParent() instanceof IASTFunctionDefinition) { IASTFunctionDeclarator dtor = ((IASTFunctionDefinition)parent.getParent()).getDeclarator(); - if( dtor instanceof IASTStandardFunctionDeclarator ) + if (dtor instanceof IASTStandardFunctionDeclarator) nodes = ((IASTStandardFunctionDeclarator)dtor).getParameters(); - else if( dtor instanceof ICASTKnRFunctionDeclarator ) + else if (dtor instanceof ICASTKnRFunctionDeclarator) nodes = ((ICASTKnRFunctionDeclarator)dtor).getParameterDeclarations(); } - if( nodes == null || nodes.length == 0 ){ + if (nodes == null || nodes.length == 0) { nodes = compound.getStatements(); } - } else if ( parent instanceof IASTTranslationUnit ){ + } else if (parent instanceof IASTTranslationUnit) { IASTTranslationUnit translation = (IASTTranslationUnit) parent; if (!prefix) { nodes = translation.getDeclarations(); @@ -1307,28 +1307,28 @@ public class CVisitor { // the AST, bug 180883 nodes = translation.getDeclarations(); } - } else if( parent instanceof IASTStandardFunctionDeclarator ){ + } else if (parent instanceof IASTStandardFunctionDeclarator) { IASTStandardFunctionDeclarator dtor = (IASTStandardFunctionDeclarator) parent; nodes = dtor.getParameters(); - scope = getContainingScope( blockItem ); - } else if( parent instanceof ICASTKnRFunctionDeclarator ){ + scope = getContainingScope(blockItem); + } else if (parent instanceof ICASTKnRFunctionDeclarator) { ICASTKnRFunctionDeclarator dtor = (ICASTKnRFunctionDeclarator) parent; nodes = dtor.getParameterDeclarations(); - scope = getContainingScope( blockItem ); + scope = getContainingScope(blockItem); } boolean typesOnly = (bits & TAGS) != 0; boolean includeBlockItem = (bits & INCLUDE_BLOCK_ITEM) != 0; - if( prefix ) + if (prefix) scope = null; - if( scope != null && ASTInternal.isFullyCached(scope) ){ + if (scope != null && ASTInternal.isFullyCached(scope)) { try { - binding = scope.getBinding( name, true, fileSet ); - } catch ( DOMException e ) { + binding = scope.getBinding(name, true, fileSet); + } catch (DOMException e) { binding = null; } - if( binding != null ) + if (binding != null) return binding; } else { if (!prefix && scope != null && scope.getParent() == null) { @@ -1345,48 +1345,48 @@ public class CVisitor { Object result = null; boolean reachedBlockItem = false; - if( nodes != null ){ + if (nodes != null) { int idx = -1; IASTNode node = nodes.length > 0 ? nodes[++idx] : null; - while( node != null ) { + while (node != null) { Object candidate = null; try { - candidate = checkForBinding( scope, node, name, typesOnly, prefixMap ); - } catch ( DOMException e ) { + candidate = checkForBinding(scope, node, name, typesOnly, prefixMap); + } catch (DOMException e) { continue; } - if( result == null && !reachedBlockItem && - ( includeBlockItem || (node != blockItem) ) ) + if (result == null && !reachedBlockItem && + (includeBlockItem || (node != blockItem))) { result = candidate; } - if( node == blockItem ){ + if (node == blockItem) { reachedBlockItem = true; } - if( idx > -1 && ++idx < nodes.length ){ + if (idx > -1 && ++idx < nodes.length) { node = nodes[idx]; } else { node = null; - if( nodes[0].getPropertyInParent() == ICASTKnRFunctionDeclarator.FUNCTION_PARAMETER || - nodes[0].getPropertyInParent() == IASTStandardFunctionDeclarator.FUNCTION_PARAMETER ) + if (nodes[0].getPropertyInParent() == ICASTKnRFunctionDeclarator.FUNCTION_PARAMETER || + nodes[0].getPropertyInParent() == IASTStandardFunctionDeclarator.FUNCTION_PARAMETER) { //function body, we were looking at parameters, now check the body itself IASTCompoundStatement compound = null; - if( parent instanceof IASTCompoundStatement ){ + if (parent instanceof IASTCompoundStatement) { compound = (IASTCompoundStatement) parent; - } else if( parent instanceof IASTFunctionDeclarator ){ + } else if (parent instanceof IASTFunctionDeclarator) { IASTNode n = parent.getParent(); - while( n instanceof IASTDeclarator ) + while (n instanceof IASTDeclarator) n = n.getParent(); - if( n instanceof IASTFunctionDefinition ){ + if (n instanceof IASTFunctionDefinition) { compound = (IASTCompoundStatement) ((IASTFunctionDefinition)n).getBody(); } } - if( compound != null ) { + if (compound != null) { nodes = compound.getStatements(); - if( nodes.length > 0 ){ + if (nodes.length > 0) { idx = 0; node = nodes[0]; } @@ -1397,38 +1397,38 @@ public class CVisitor { } else { try { - result = checkForBinding( scope, parent, name, typesOnly, prefixMap ); - } catch ( DOMException e ) { + result = checkForBinding(scope, parent, name, typesOnly, prefixMap); + } catch (DOMException e) { } } - if( scope != null ) { + if (scope != null) { try { ASTInternal.setFullyCached(scope, true); - } catch ( DOMException e ) { + } catch (DOMException e) { } } - if( result != null ){ - if(CVisitor.declaredBefore((IASTName)result, name)) { + if (result != null) { + if (CVisitor.declaredBefore((IASTName)result, name)) { return ((IASTName)result).resolveBinding(); } } } - if( (bits & CURRENT_SCOPE) == 0 ) + if ((bits & CURRENT_SCOPE) == 0) blockItem = parent; else blockItem = null; - if( blockItem instanceof IASTTranslationUnit ) + if (blockItem instanceof IASTTranslationUnit) break; } if (foundIndexBinding != null) { return foundIndexBinding; } - if( prefixMap != null ){ - IBinding [] result = null; - Object [] vals = prefixMap.valueArray(); + if (prefixMap != null) { + IBinding[] result = null; + Object[] vals = prefixMap.valueArray(); for (Object val : vals) { - result = (IBinding[]) ArrayUtil.append( IBinding.class, result, ((IASTName) val).resolveBinding() ); + result = (IBinding[]) ArrayUtil.append(IBinding.class, result, ((IASTName) val).resolveBinding()); } IASTTranslationUnit tu = (IASTTranslationUnit)blockItem; @@ -1446,40 +1446,40 @@ public class CVisitor { } } - return ArrayUtil.trim( IBinding.class, result ); + return ArrayUtil.trim(IBinding.class, result); } - if( blockItem != null) { + if (blockItem != null) { if (binding == null) - return externalBinding( (IASTTranslationUnit) blockItem, name ); + return externalBinding((IASTTranslationUnit) blockItem, name); return binding; } return null; } - private static IBinding externalBinding( IASTTranslationUnit tu, IASTName name ){ + private static IBinding externalBinding(IASTTranslationUnit tu, IASTName name) { IASTNode parent = name.getParent(); IBinding external = null; - if( parent instanceof IASTIdExpression ){ - if( parent.getPropertyInParent() == IASTFunctionCallExpression.FUNCTION_NAME ){ + if (parent instanceof IASTIdExpression) { + if (parent.getPropertyInParent() == IASTFunctionCallExpression.FUNCTION_NAME) { //external function - external = new CExternalFunction( tu, name ); - ((CScope)tu.getScope()).addName( name ); + external = new CExternalFunction(tu, name); + ((CScope)tu.getScope()).addName(name); } else { //external variable - //external = new CExternalVariable( tu, name ); - //((CScope)tu.getScope()).addName( name ); - external = new ProblemBinding( name, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray() ); + //external = new CExternalVariable(tu, name); + //((CScope)tu.getScope()).addName(name); + external = new ProblemBinding(name, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray()); } } return external; } - private static IASTName checkForBinding( IScope scope, IASTDeclSpecifier declSpec, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap ) throws DOMException{ + private static IASTName checkForBinding(IScope scope, IASTDeclSpecifier declSpec, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap) throws DOMException{ IASTName tempName = null; IASTName resultName = null; - char [] n = name.toCharArray(); - if( declSpec instanceof ICASTElaboratedTypeSpecifier ){ + char[] n = name.toCharArray(); + if (declSpec instanceof ICASTElaboratedTypeSpecifier) { tempName = ((ICASTElaboratedTypeSpecifier)declSpec).getName(); // Don't include the query name in the results @@ -1487,60 +1487,60 @@ public class CVisitor { return null; } - if( scope != null ) - ASTInternal.addName( scope, tempName ); - if( typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (scope != null) + ASTInternal.addName(scope, tempName); + if (typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } - } else if( declSpec instanceof ICASTCompositeTypeSpecifier ){ + } else if (declSpec instanceof ICASTCompositeTypeSpecifier) { tempName = ((ICASTCompositeTypeSpecifier)declSpec).getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); + if (scope != null) + ASTInternal.addName(scope, tempName); - if( typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } //also have to check for any nested structs - IASTDeclaration [] nested = ((ICASTCompositeTypeSpecifier)declSpec).getMembers(); + IASTDeclaration[] nested = ((ICASTCompositeTypeSpecifier)declSpec).getMembers(); for (IASTDeclaration element : nested) { - if( element instanceof IASTSimpleDeclaration ){ + if (element instanceof IASTSimpleDeclaration) { IASTDeclSpecifier d = ((IASTSimpleDeclaration)element).getDeclSpecifier(); - if( d instanceof ICASTCompositeTypeSpecifier || d instanceof IASTEnumerationSpecifier ) { - Object obj = checkForBinding( scope, d, name, typesOnly, prefixMap ); - if( prefixMap == null && resultName == null ){ + if (d instanceof ICASTCompositeTypeSpecifier || d instanceof IASTEnumerationSpecifier) { + Object obj = checkForBinding(scope, d, name, typesOnly, prefixMap); + if (prefixMap == null && resultName == null) { resultName = (IASTName) obj; } } } } - } else if( declSpec instanceof ICASTEnumerationSpecifier ){ + } else if (declSpec instanceof ICASTEnumerationSpecifier) { ICASTEnumerationSpecifier enumeration = (ICASTEnumerationSpecifier) declSpec; tempName = enumeration.getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); - if( typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (scope != null) + ASTInternal.addName(scope, tempName); + if (typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } //check enumerators - IASTEnumerator [] list = ((ICASTEnumerationSpecifier) declSpec).getEnumerators(); + IASTEnumerator[] list = ((ICASTEnumerationSpecifier) declSpec).getEnumerators(); for (IASTEnumerator enumerator : list) { - if( enumerator == null ) break; + if (enumerator == null) break; tempName = enumerator.getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); - if( !typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (scope != null) + ASTInternal.addName(scope, tempName); + if (!typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } } @@ -1548,36 +1548,36 @@ public class CVisitor { return resultName; } - private static Object collectResult( IASTName candidate, char[] name, CharArrayObjectMap prefixMap ){ - char [] c = candidate.toCharArray(); - if( prefixMap == null && CharArrayUtils.equals( c, name ) ){ + private static Object collectResult(IASTName candidate, char[] name, CharArrayObjectMap prefixMap) { + char[] c = candidate.toCharArray(); + if (prefixMap == null && CharArrayUtils.equals(c, name)) { return candidate; - } else if( prefixMap != null && CharArrayUtils.equals( c, 0, name.length, name, true ) && !prefixMap.containsKey( c ) ){ - prefixMap.put( c, candidate ); + } else if (prefixMap != null && CharArrayUtils.equals(c, 0, name.length, name, true) && !prefixMap.containsKey(c)) { + prefixMap.put(c, candidate); } return prefixMap; } - private static IASTName checkForBinding( IScope scope, IASTParameterDeclaration paramDecl, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap ) throws DOMException{ - if( paramDecl == null ) return null; + private static IASTName checkForBinding(IScope scope, IASTParameterDeclaration paramDecl, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap) throws DOMException{ + if (paramDecl == null) return null; IASTDeclarator dtor = paramDecl.getDeclarator(); - while( dtor.getNestedDeclarator() != null ){ + while (dtor.getNestedDeclarator() != null) { dtor = dtor.getNestedDeclarator(); } IASTName tempName = dtor.getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); + if (scope != null) + ASTInternal.addName(scope, tempName); - if( !typesOnly ) { - char [] c = tempName.toCharArray(); - char [] n = name.toCharArray(); - if( prefixMap == null && CharArrayUtils.equals( c, n ) ) + if (!typesOnly) { + char[] c = tempName.toCharArray(); + char[] n = name.toCharArray(); + if (prefixMap == null && CharArrayUtils.equals(c, n)) return tempName; - else if( prefixMap != null && CharArrayUtils.equals( c, 0, n.length, n, true ) && !prefixMap.containsKey( c ) ) - prefixMap.put( c, tempName ); + else if (prefixMap != null && CharArrayUtils.equals(c, 0, n.length, n, true) && !prefixMap.containsKey(c)) + prefixMap.put(c, tempName); } else { - return checkForBinding( scope, paramDecl.getDeclSpecifier(), name, typesOnly, prefixMap ); + return checkForBinding(scope, paramDecl.getDeclSpecifier(), name, typesOnly, prefixMap); } return null; } @@ -1586,65 +1586,66 @@ public class CVisitor { * if not a prefix lookup, returns IASTName * if doing prefix lookup, results are in prefixMap, returns null */ - private static IASTName checkForBinding( IScope scope, IASTNode node, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap ) throws DOMException{ - if( node instanceof IASTDeclaration ){ - return checkForBinding( scope, (IASTDeclaration) node, name, typesOnly, prefixMap ); - } else if( node instanceof IASTParameterDeclaration ){ - return checkForBinding( scope, (IASTParameterDeclaration) node, name, typesOnly, prefixMap ); - } else if( node instanceof IASTDeclarationStatement ){ - return checkForBinding( scope, ((IASTDeclarationStatement)node).getDeclaration(), name, typesOnly, prefixMap ); - } else if( node instanceof IASTForStatement ){ + private static IASTName checkForBinding(IScope scope, IASTNode node, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap) throws DOMException{ + if (node instanceof IASTDeclaration) { + return checkForBinding(scope, (IASTDeclaration) node, name, typesOnly, prefixMap); + } else if (node instanceof IASTParameterDeclaration) { + return checkForBinding(scope, (IASTParameterDeclaration) node, name, typesOnly, prefixMap); + } else if (node instanceof IASTDeclarationStatement) { + return checkForBinding(scope, ((IASTDeclarationStatement)node).getDeclaration(), name, typesOnly, prefixMap); + } else if (node instanceof IASTForStatement) { IASTForStatement forStatement = (IASTForStatement) node; - if( forStatement.getInitializerStatement() instanceof IASTDeclarationStatement ){ - return checkForBinding( scope, ((IASTDeclarationStatement)forStatement.getInitializerStatement()).getDeclaration(), name, typesOnly, prefixMap ); + if (forStatement.getInitializerStatement() instanceof IASTDeclarationStatement) { + return checkForBinding(scope, ((IASTDeclarationStatement)forStatement.getInitializerStatement()).getDeclaration(), name, typesOnly, prefixMap); } } return null; } - private static IASTName checkForBinding( IScope scope, IASTDeclaration declaration, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap ) throws DOMException{ - char [] n = name.toCharArray(); + + private static IASTName checkForBinding(IScope scope, IASTDeclaration declaration, IASTName name, boolean typesOnly, CharArrayObjectMap prefixMap) throws DOMException{ + char[] n = name.toCharArray(); IASTName tempName = null; IASTName resultName = null; - if( declaration instanceof IASTSimpleDeclaration ){ + if (declaration instanceof IASTSimpleDeclaration) { IASTSimpleDeclaration simpleDeclaration = (IASTSimpleDeclaration) declaration; - IASTDeclarator [] declarators = simpleDeclaration.getDeclarators(); + IASTDeclarator[] declarators = simpleDeclaration.getDeclarators(); for (IASTDeclarator declarator : declarators) { - while( declarator.getNestedDeclarator() != null ){ + while (declarator.getNestedDeclarator() != null) { declarator = declarator.getNestedDeclarator(); } tempName = declarator.getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); + if (scope != null) + ASTInternal.addName(scope, tempName); - if( !typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (!typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } } - tempName = checkForBinding( scope, simpleDeclaration.getDeclSpecifier(), name, typesOnly, prefixMap ); - if( prefixMap == null && tempName != null ){ + tempName = checkForBinding(scope, simpleDeclaration.getDeclSpecifier(), name, typesOnly, prefixMap); + if (prefixMap == null && tempName != null) { resultName = tempName; } - } else if( !typesOnly && declaration instanceof IASTFunctionDefinition ){ + } else if (!typesOnly && declaration instanceof IASTFunctionDefinition) { IASTFunctionDefinition functionDef = (IASTFunctionDefinition) declaration; IASTDeclarator dtor = functionDef.getDeclarator(); tempName = dtor.getName(); - if( scope != null ) - ASTInternal.addName( scope, tempName ); + if (scope != null) + ASTInternal.addName(scope, tempName); - if( !typesOnly ){ - if( prefixMap != null ) - prefixMap = (CharArrayObjectMap) collectResult( tempName, n, prefixMap ); - else if( collectResult( tempName, n, prefixMap ) != null ) + if (!typesOnly) { + if (prefixMap != null) + prefixMap = (CharArrayObjectMap) collectResult(tempName, n, prefixMap); + else if (collectResult(tempName, n, prefixMap) != null) resultName = tempName; } - tempName = checkForBinding( scope, functionDef.getDeclSpecifier(), name, typesOnly, prefixMap ); - if( prefixMap == null && tempName != null ){ + tempName = checkForBinding(scope, functionDef.getDeclSpecifier(), name, typesOnly, prefixMap); + if (prefixMap == null && tempName != null) { resultName = tempName; } } @@ -1652,66 +1653,68 @@ public class CVisitor { return resultName; } - protected static IASTDeclarator findDefinition( IASTDeclarator declarator, int beginAtLoc ){ - return (IASTDeclarator) findDefinition( declarator, declarator.getName().toCharArray(), beginAtLoc ); + protected static IASTDeclarator findDefinition(IASTDeclarator declarator, int beginAtLoc) { + return (IASTDeclarator) findDefinition(declarator, declarator.getName().toCharArray(), beginAtLoc); } - protected static IASTFunctionDeclarator findDefinition( IASTFunctionDeclarator declarator ){ - return (IASTFunctionDeclarator) findDefinition( declarator, declarator.getName().toCharArray(), AT_NEXT ); + + protected static IASTFunctionDeclarator findDefinition(IASTFunctionDeclarator declarator) { + return (IASTFunctionDeclarator) findDefinition(declarator, declarator.getName().toCharArray(), AT_NEXT); } - protected static IASTDeclSpecifier findDefinition( ICASTElaboratedTypeSpecifier declSpec ){ + + protected static IASTDeclSpecifier findDefinition(ICASTElaboratedTypeSpecifier declSpec) { return (IASTDeclSpecifier) findDefinition(declSpec, declSpec.getName().toCharArray(), AT_BEGINNING); } - private static IASTNode findDefinition(IASTNode decl, char [] declName, int beginAtLoc) { - IASTNode blockItem = getContainingBlockItem( decl ); + private static IASTNode findDefinition(IASTNode decl, char[] declName, int beginAtLoc) { + IASTNode blockItem = getContainingBlockItem(decl); IASTNode parent = blockItem.getParent(); - IASTNode [] list = null; - if( parent instanceof IASTCompoundStatement ){ + IASTNode[] list = null; + if (parent instanceof IASTCompoundStatement) { IASTCompoundStatement compound = (IASTCompoundStatement) parent; list = compound.getStatements(); - } else if ( parent instanceof IASTTranslationUnit ){ + } else if (parent instanceof IASTTranslationUnit) { IASTTranslationUnit translation = (IASTTranslationUnit) parent; list = translation.getDeclarations(); } - boolean begun = ( beginAtLoc == AT_BEGINNING ); - if( list != null ){ + boolean begun = (beginAtLoc == AT_BEGINNING); + if (list != null) { for (IASTNode node : list) { - if( node == blockItem ){ + if (node == blockItem) { begun = true; continue; } - if( begun ) { - if( node instanceof IASTDeclarationStatement ){ + if (begun) { + if (node instanceof IASTDeclarationStatement) { node = ((IASTDeclarationStatement) node).getDeclaration(); } - if( node instanceof IASTFunctionDefinition && decl instanceof IASTFunctionDeclarator ){ + if (node instanceof IASTFunctionDefinition && decl instanceof IASTFunctionDeclarator) { IASTFunctionDeclarator dtor = ((IASTFunctionDefinition) node).getDeclarator(); IASTName name = dtor.getName(); - if( name.toString().equals( declName )){ + if (name.toString().equals(declName)) { return dtor; } - } else if( node instanceof IASTSimpleDeclaration && decl instanceof ICASTElaboratedTypeSpecifier){ + } else if (node instanceof IASTSimpleDeclaration && decl instanceof ICASTElaboratedTypeSpecifier) { IASTSimpleDeclaration simpleDecl = (IASTSimpleDeclaration) node; IASTDeclSpecifier declSpec = simpleDecl.getDeclSpecifier(); IASTName name = null; - if( declSpec instanceof ICASTCompositeTypeSpecifier ){ + if (declSpec instanceof ICASTCompositeTypeSpecifier) { name = ((ICASTCompositeTypeSpecifier)declSpec).getName(); - } else if( declSpec instanceof ICASTEnumerationSpecifier ){ + } else if (declSpec instanceof ICASTEnumerationSpecifier) { name = ((ICASTEnumerationSpecifier)declSpec).getName(); } - if( name != null ){ - if( CharArrayUtils.equals( name.toCharArray(), declName ) ){ + if (name != null) { + if (CharArrayUtils.equals(name.toCharArray(), declName)) { return declSpec; } } - } else if( node instanceof IASTSimpleDeclaration && decl instanceof IASTDeclarator ){ + } else if (node instanceof IASTSimpleDeclaration && decl instanceof IASTDeclarator) { IASTSimpleDeclaration simpleDecl = (IASTSimpleDeclaration) node; - IASTDeclarator [] dtors = simpleDecl.getDeclarators(); - for( int j = 0; dtors != null && j < dtors.length; j++ ){ - if( CharArrayUtils.equals( dtors[j].getName().toCharArray(), declName ) ){ + IASTDeclarator[] dtors = simpleDecl.getDeclarators(); + for (int j = 0; dtors != null && j < dtors.length; j++) { + if (CharArrayUtils.equals(dtors[j].getName().toCharArray(), declName)) { return dtors[j]; } } @@ -1722,8 +1725,8 @@ public class CVisitor { return null; } - public static void clearBindings( IASTTranslationUnit tu ){ - tu.accept( new ClearBindingAction() ); + public static void clearBindings(IASTTranslationUnit tu) { + tu.accept(new ClearBindingAction()); } /** @@ -1736,75 +1739,75 @@ public class CVisitor { IASTDeclSpecifier declSpec = null; IASTNode node = declarator.getParent(); - while( node instanceof IASTDeclarator ){ + while (node instanceof IASTDeclarator) { declarator = (IASTDeclarator) node; node = node.getParent(); } - if( node instanceof IASTParameterDeclaration ) + if (node instanceof IASTParameterDeclaration) declSpec = ((IASTParameterDeclaration) node).getDeclSpecifier(); - else if( node instanceof IASTSimpleDeclaration ) + else if (node instanceof IASTSimpleDeclaration) declSpec = ((IASTSimpleDeclaration)node).getDeclSpecifier(); - else if( node instanceof IASTFunctionDefinition ) + else if (node instanceof IASTFunctionDefinition) declSpec = ((IASTFunctionDefinition)node).getDeclSpecifier(); - else if( node instanceof IASTTypeId ) + else if (node instanceof IASTTypeId) declSpec = ((IASTTypeId)node).getDeclSpecifier(); - boolean isParameter = ( node instanceof IASTParameterDeclaration || node.getParent() instanceof ICASTKnRFunctionDeclarator ); + boolean isParameter = (node instanceof IASTParameterDeclaration || node.getParent() instanceof ICASTKnRFunctionDeclarator); IType type = null; //C99 6.7.5.3-12 The storage class specifier for a parameter declaration is ignored unless the declared parameter is one of the //members of the parameter type list for a function definition. - if( isParameter && node.getParent().getParent() instanceof IASTFunctionDefinition ){ - type = createBaseType( declSpec ); + if (isParameter && node.getParent().getParent() instanceof IASTFunctionDefinition) { + type = createBaseType(declSpec); } else { - type = createType( (ICASTDeclSpecifier) declSpec ); + type = createType((ICASTDeclSpecifier) declSpec); } - type = createType( type, declarator ); + type = createType(type, declarator); - if( isParameter ) { + if (isParameter) { //C99: 6.7.5.3-7 a declaration of a parameter as "array of type" shall be adjusted to "qualified pointer to type", where the - //type qualifiers (if any) are those specified within the [ and ] of the array type derivation - if( type instanceof IArrayType ){ + //type qualifiers (if any) are those specified within the[and] of the array type derivation + if (type instanceof IArrayType) { CArrayType at = (CArrayType) type; - type = new CQualifiedPointerType( at.getType(), at.getModifier() ); - } else if( type instanceof IFunctionType ) { + type = new CQualifiedPointerType(at.getType(), at.getModifier()); + } else if (type instanceof IFunctionType) { //-8 A declaration of a parameter as "function returning type" shall be adjusted to "pointer to function returning type" - type = new CPointerType( type, 0 ); + type = new CPointerType(type, 0); } } return type; } - public static IType createType( IType baseType, IASTDeclarator declarator ) { - if( declarator instanceof IASTFunctionDeclarator ) - return createType( baseType, (IASTFunctionDeclarator)declarator ); + public static IType createType(IType baseType, IASTDeclarator declarator) { + if (declarator instanceof IASTFunctionDeclarator) + return createType(baseType, (IASTFunctionDeclarator)declarator); IType type = baseType; - type = setupPointerChain( declarator.getPointerOperators(), type ); - type = setupArrayChain( declarator, type ); + type = setupPointerChain(declarator.getPointerOperators(), type); + type = setupArrayChain(declarator, type); IASTDeclarator nested = declarator.getNestedDeclarator(); - if( nested != null ) { - return createType( type, nested ); + if (nested != null) { + return createType(type, nested); } return type; } - public static IType createType( IType returnType, IASTFunctionDeclarator declarator){ + public static IType createType(IType returnType, IASTFunctionDeclarator declarator) { - IType [] pTypes = getParmTypes( declarator ); - returnType = setupPointerChain( declarator.getPointerOperators(), returnType ); + IType[] pTypes = getParmTypes(declarator); + returnType = setupPointerChain(declarator.getPointerOperators(), returnType); - IType type = new CFunctionType( returnType, pTypes ); + IType type = new CFunctionType(returnType, pTypes); IASTDeclarator nested = declarator.getNestedDeclarator(); - if( nested != null ) { - return createType( type, nested ); + if (nested != null) { + return createType(type, nested); } return type; } @@ -1817,68 +1820,69 @@ public class CVisitor { * @param declSpec the IASTDeclSpecifier used to determine if the base type is a CQualifierType or not * @return the base IType */ - public static IType createBaseType( IASTDeclSpecifier declSpec ) { - if( declSpec instanceof IGCCASTSimpleDeclSpecifier ){ + public static IType createBaseType(IASTDeclSpecifier declSpec) { + if (declSpec instanceof IGCCASTSimpleDeclSpecifier) { IASTExpression exp = ((IGCCASTSimpleDeclSpecifier)declSpec).getTypeofExpression(); - if( exp != null ) - return getExpressionType( exp ); - return new CBasicType( (ICASTSimpleDeclSpecifier) declSpec ); + if (exp != null) + return getExpressionType(exp); + return new CBasicType((ICASTSimpleDeclSpecifier) declSpec); } else if (declSpec instanceof ICASTSimpleDeclSpecifier) { return new CBasicType((ICASTSimpleDeclSpecifier)declSpec); } IBinding binding = null; IASTName name = null; - if( declSpec instanceof ICASTTypedefNameSpecifier ){ + if (declSpec instanceof ICASTTypedefNameSpecifier) { name = ((ICASTTypedefNameSpecifier) declSpec).getName(); - } else if( declSpec instanceof IASTElaboratedTypeSpecifier ){ + } else if (declSpec instanceof IASTElaboratedTypeSpecifier) { name = ((IASTElaboratedTypeSpecifier) declSpec).getName(); - } else if( declSpec instanceof IASTCompositeTypeSpecifier ){ + } else if (declSpec instanceof IASTCompositeTypeSpecifier) { name = ((IASTCompositeTypeSpecifier) declSpec).getName(); - } else if( declSpec instanceof IASTEnumerationSpecifier ){ + } else if (declSpec instanceof IASTEnumerationSpecifier) { name = ((IASTEnumerationSpecifier)declSpec).getName(); } else { - return new ProblemBinding(declSpec, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, declSpec.getRawSignature().toCharArray() ); + return new ProblemBinding(declSpec, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, declSpec.getRawSignature().toCharArray()); } binding = name.resolveBinding(); - if( binding instanceof IType ) + if (binding instanceof IType) return (IType) binding; - if( binding != null ) - return new ProblemBinding( name, IProblemBinding.SEMANTIC_INVALID_TYPE, name.toCharArray() ); - return new ProblemBinding( name, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray() ); + if (binding != null) + return new ProblemBinding(name, IProblemBinding.SEMANTIC_INVALID_TYPE, name.toCharArray()); + return new ProblemBinding(name, IProblemBinding.SEMANTIC_NAME_NOT_FOUND, name.toCharArray()); } - public static IType createType( ICASTDeclSpecifier declSpec ) { + public static IType createType(ICASTDeclSpecifier declSpec) { if (declSpec.isConst() || declSpec.isVolatile() || declSpec.isRestrict()) { return new CQualifierType(declSpec); } - return createBaseType( declSpec ); + return createBaseType(declSpec); } + /** * Returns an IType[] corresponding to the parameter types of the IASTFunctionDeclarator parameter. * * @param decltor the IASTFunctionDeclarator to create an IType[] for its parameters * @return IType[] corresponding to the IASTFunctionDeclarator parameters */ - private static IType[] getParmTypes( IASTFunctionDeclarator decltor ){ - if ( decltor instanceof IASTStandardFunctionDeclarator ) { + private static IType[] getParmTypes(IASTFunctionDeclarator decltor) { + if (decltor instanceof IASTStandardFunctionDeclarator) { IASTParameterDeclaration parms[] = ((IASTStandardFunctionDeclarator)decltor).getParameters(); IType parmTypes[] = new IType[parms.length]; - for( int i = 0; i < parms.length; i++ ){ - parmTypes[i] = createType( parms[i].getDeclarator() ); + for (int i = 0; i < parms.length; i++) { + parmTypes[i] = createType(parms[i].getDeclarator()); } return parmTypes; - } else if ( decltor instanceof ICASTKnRFunctionDeclarator ) { + } else if (decltor instanceof ICASTKnRFunctionDeclarator) { IASTName parms[] = ((ICASTKnRFunctionDeclarator)decltor).getParameterNames(); IType parmTypes[] = new IType[parms.length]; - for( int i = 0; i < parms.length; i++ ){ - IASTDeclarator dtor = getKnRParameterDeclarator( (ICASTKnRFunctionDeclarator) decltor, parms[i] ); - if( dtor != null ) - parmTypes[i] = createType( dtor ); + for (int i = 0; i < parms.length; i++) { + IASTDeclarator dtor = getKnRParameterDeclarator((ICASTKnRFunctionDeclarator) decltor, parms[i]); + if (dtor != null) + parmTypes[i] = createType(dtor); } return parmTypes; } else { @@ -1886,16 +1890,16 @@ public class CVisitor { } } - protected static IASTDeclarator getKnRParameterDeclarator( ICASTKnRFunctionDeclarator fKnRDtor, IASTName name ){ - IASTDeclaration [] decls = fKnRDtor.getParameterDeclarations(); - char [] n = name.toCharArray(); - for( int i = 0; i < decls.length; i++ ){ - if( !( decls[i] instanceof IASTSimpleDeclaration ) ) + protected static IASTDeclarator getKnRParameterDeclarator(ICASTKnRFunctionDeclarator fKnRDtor, IASTName name) { + IASTDeclaration[] decls = fKnRDtor.getParameterDeclarations(); + char[] n = name.toCharArray(); + for (int i = 0; i < decls.length; i++) { + if (!(decls[i] instanceof IASTSimpleDeclaration)) continue; - IASTDeclarator [] dtors = ((IASTSimpleDeclaration)decls[i]).getDeclarators(); + IASTDeclarator[] dtors = ((IASTSimpleDeclaration)decls[i]).getDeclarators(); for (IASTDeclarator dtor : dtors) { - if( CharArrayUtils.equals( dtor.getName().toCharArray(), n ) ){ + if (CharArrayUtils.equals(dtor.getName().toCharArray(), n)) { return dtor; } } @@ -1945,7 +1949,7 @@ public class CVisitor { private static IType setupPointerChain(IASTPointerOperator[] ptrs, IType lastType) { CPointerType pointerType = null; - if ( ptrs != null && ptrs.length > 0 ) { + if (ptrs != null && ptrs.length > 0) { pointerType = new CPointerType(); if (ptrs.length == 1) { @@ -1997,68 +2001,64 @@ public class CVisitor { return action.getDeclarationNames(); } - public static IASTName[] getReferences(IASTTranslationUnit tu, IBinding binding) { - CollectReferencesAction action = new CollectReferencesAction( binding ); + CollectReferencesAction action = new CollectReferencesAction(binding); tu.accept(action); return action.getReferences(); } - public static IBinding findTypeBinding(IASTNode startingPoint, IASTName name) throws DOMException { - if( startingPoint instanceof IASTTranslationUnit ) - { - IASTDeclaration [] declarations = ((IASTTranslationUnit)startingPoint).getDeclarations(); - if( declarations.length > 0 ) - return (IBinding) findBinding( declarations[declarations.length - 1], name, COMPLETE | INCLUDE_BLOCK_ITEM ); + if (startingPoint instanceof IASTTranslationUnit) { + IASTDeclaration[] declarations = ((IASTTranslationUnit)startingPoint).getDeclarations(); + if (declarations.length > 0) + return (IBinding) findBinding(declarations[declarations.length - 1], name, COMPLETE | INCLUDE_BLOCK_ITEM ); } - if( startingPoint instanceof IASTCompoundStatement ) - { - IASTStatement [] statements = ((IASTCompoundStatement)startingPoint).getStatements(); - if( statements.length > 0 ) - return (IBinding) findBinding( statements[ statements.length - 1 ], name, COMPLETE | INCLUDE_BLOCK_ITEM ); + if (startingPoint instanceof IASTCompoundStatement) { + IASTStatement[] statements = ((IASTCompoundStatement)startingPoint).getStatements(); + if (statements.length > 0) + return (IBinding) findBinding(statements[statements.length - 1], name, COMPLETE | INCLUDE_BLOCK_ITEM); } return null; } - public static IBinding [] findBindingsForContentAssist( IASTName name, boolean isPrefix ){ + public static IBinding[] findBindingsForContentAssist(IASTName name, boolean isPrefix) { ASTNodeProperty prop = name.getPropertyInParent(); - IBinding [] result = null; + IBinding[] result = null; - if( prop == IASTFieldReference.FIELD_NAME ){ - result = (IBinding[]) findBinding( (IASTFieldReference) name.getParent(), isPrefix ); + if (prop == IASTFieldReference.FIELD_NAME) { + result = (IBinding[]) findBinding((IASTFieldReference) name.getParent(), isPrefix); } else { int bits = isPrefix ? PREFIX_LOOKUP : COMPLETE; - if( prop == IASTElaboratedTypeSpecifier.TYPE_NAME ){ + if (prop == IASTElaboratedTypeSpecifier.TYPE_NAME) { bits |= TAGS; - } else if( prop == IASTIdExpression.ID_NAME ){ + } else if (prop == IASTIdExpression.ID_NAME) { bits |= INCLUDE_BLOCK_ITEM; } - IASTNode blockItem = getContainingBlockItem( name ); + IASTNode blockItem = getContainingBlockItem(name); try { - result = isPrefix ? (IBinding[]) findBinding( blockItem, name, bits ) : - new IBinding[] { (IBinding) findBinding( blockItem, name, bits ) }; - } catch ( DOMException e ) { + result = isPrefix ? (IBinding[]) findBinding(blockItem, name, bits) : + new IBinding[] { (IBinding) findBinding(blockItem, name, bits) }; + } catch (DOMException e) { } } - return (IBinding[]) ArrayUtil.trim( IBinding.class, result ); + return (IBinding[]) ArrayUtil.trim(IBinding.class, result); } - public static IBinding[] findBindings( IScope scope, String name, boolean prefixLookup ) throws DOMException{ + public static IBinding[] findBindings(IScope scope, String name, boolean prefixLookup) throws DOMException { IASTNode node = ASTInternal.getPhysicalNodeOfScope(scope); - if( node instanceof IASTFunctionDefinition ) + if (node instanceof IASTFunctionDefinition) node = ((IASTFunctionDefinition)node).getBody(); - CASTName astName = new CASTName( name.toCharArray() ); - astName.setParent( node ); + CASTName astName = new CASTName(name.toCharArray()); + astName.setParent(node); //normal names - astName.setPropertyInParent( STRING_LOOKUP_PROPERTY ); + astName.setPropertyInParent(STRING_LOOKUP_PROPERTY); int flags = prefixLookup ? COMPLETE | PREFIX_LOOKUP : COMPLETE; - Object o1 = findBinding( astName, astName, flags ); + Object o1 = findBinding(astName, astName, flags); IBinding[] b1 = null; if (o1 instanceof IBinding) { @@ -2068,9 +2068,9 @@ public class CVisitor { } //structure names - astName.setPropertyInParent( STRING_LOOKUP_TAGS_PROPERTY ); + astName.setPropertyInParent(STRING_LOOKUP_TAGS_PROPERTY); flags = prefixLookup ? COMPLETE | TAGS | PREFIX_LOOKUP : COMPLETE | TAGS; - Object o2 = findBinding( astName, astName, flags ); + Object o2 = findBinding(astName, astName, flags); IBinding[] b2 = null; if (o2 instanceof IBinding) { @@ -2081,10 +2081,10 @@ public class CVisitor { //label names List b3 = new ArrayList(); - do{ - char [] n = name.toCharArray(); - if( scope instanceof ICFunctionScope ){ - ILabel [] labels = ((CFunctionScope)scope).getLabels(); + do { + char[] n = name.toCharArray(); + if (scope instanceof ICFunctionScope) { + ILabel[] labels = ((CFunctionScope)scope).getLabels(); for (ILabel label : labels) { if (prefixLookup) { if (CharArrayUtils.equals(label.getNameCharArray(), @@ -2101,11 +2101,11 @@ public class CVisitor { if (!prefixLookup) break; } scope = scope.getParent(); - } while( scope != null ); + } while (scope != null); int c = (b1 == null ? 0 : b1.length) + (b2 == null ? 0 : b2.length) + b3.size(); - IBinding [] result = new IBinding [c]; + IBinding[] result = new IBinding[c]; if (b1 != null) ArrayUtil.addAll(IBinding.class, result, b1); @@ -2118,39 +2118,40 @@ public class CVisitor { return result; } - static public boolean declaredBefore(IASTNode nodeA, IASTNode nodeB){ - if( nodeB == null ) return true; - if( nodeB.getPropertyInParent() == STRING_LOOKUP_PROPERTY ) return true; + static public boolean declaredBefore(IASTNode nodeA, IASTNode nodeB) { + if (nodeB == null) return true; + if (nodeB.getPropertyInParent() == STRING_LOOKUP_PROPERTY) return true; - if(nodeA instanceof ASTNode) { + if (nodeA instanceof ASTNode) { ASTNode nd= (ASTNode) nodeA; int pointOfDecl = 0; ASTNodeProperty prop = nd.getPropertyInParent(); //point of declaration for a name is immediately after its complete declarator and before its initializer - if( prop == IASTDeclarator.DECLARATOR_NAME || nd instanceof IASTDeclarator ){ + if (prop == IASTDeclarator.DECLARATOR_NAME || nd instanceof IASTDeclarator) { IASTDeclarator dtor = (IASTDeclarator)((nd instanceof IASTDeclarator) ? nd : nd.getParent()); - while( dtor.getParent() instanceof IASTDeclarator ) + while (dtor.getParent() instanceof IASTDeclarator) dtor = (IASTDeclarator) dtor.getParent(); IASTInitializer init = dtor.getInitializer(); - if( init != null ) + if (init != null) pointOfDecl = ((ASTNode)init).getOffset() - 1; else pointOfDecl = ((ASTNode)dtor).getOffset() + ((ASTNode)dtor).getLength(); } //point of declaration for an enumerator is immediately after it enumerator-definition - else if( prop == IASTEnumerator.ENUMERATOR_NAME) { + else if (prop == IASTEnumerator.ENUMERATOR_NAME) { IASTEnumerator enumtor = (IASTEnumerator) nd.getParent(); - if( enumtor.getValue() != null ){ + if (enumtor.getValue() != null) { ASTNode exp = (ASTNode) enumtor.getValue(); pointOfDecl = exp.getOffset() + exp.getLength(); } else { pointOfDecl = nd.getOffset() + nd.getLength(); } - } else + } else { pointOfDecl = nd.getOffset() + nd.getLength(); + } - return ( pointOfDecl < ((ASTNode)nodeB).getOffset() ); + return (pointOfDecl < ((ASTNode)nodeB).getOffset()); } return true;