mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-23 14:42:11 +02:00
Code Issues Wiki Activity

fix bug 89851

Browse source 
when considering the offset of a binding, use the lowest offset of all its declarations/definitions
instead of just the offset of the declaration that was resolved first.
This commit is contained in:
Andrew Niefer 2005-03-31 20:24:13 +00:00
parent 9a6a9c9350
commit cc1cf61ddf
2 changed files with 34 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2CPPTests.java
View file

@ -3120,5 +3120,23 @@ public class AST2CPPTests extends AST2BaseTest {
assertTrue( col.getName(4).resolveBinding() instanceof ICPPConstructor );
}
public void testBug89851() throws Exception {
StringBuffer buffer = new StringBuffer();
buffer.append( "class B * b; \n"); //$NON-NLS-1$
buffer.append( "class A { \n"); //$NON-NLS-1$
buffer.append( " A * a; \n"); //$NON-NLS-1$
buffer.append( "}; \n"); //$NON-NLS-1$
buffer.append( "class A; \n"); //$NON-NLS-1$
IASTTranslationUnit tu = parse( buffer.toString(), ParserLanguage.CPP ); //$NON-NLS-1$
CPPNameCollector col = new CPPNameCollector();
tu.accept( col );
assertTrue( col.getName(0).resolveBinding() instanceof ICPPClassType );
assertTrue( col.getName(1).resolveBinding() instanceof ICPPVariable );
assertTrue( col.getName(2).resolveBinding() instanceof ICPPClassType );
assertTrue( col.getName(3).resolveBinding() instanceof ICPPClassType );
}
}

18
core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPSemantics.java
View file

@ -1318,12 +1318,22 @@ public class CPPSemantics {
if( obj instanceof ICPPInternalBinding ){
ICPPInternalBinding cpp = (ICPPInternalBinding) obj;
IASTNode[] n = cpp.getDeclarations();
if( n != null && n.length > 0 )
int o, offset = -1;
if( n != null && n.length > 0 ) {
nd = (ASTNode) n[0];
else if( cpp.getDefinition() != null )
offset = ((ASTNode) n[0]).getOffset();
for (int i = 1; i < n.length && n[i] != null; i++) {
o = ((ASTNode) n[i]).getOffset();
if( o < offset )
nd = (ASTNode) n[i];
}
}
if( cpp.getDefinition() != null ){
if( nd == null || ((ASTNode)cpp.getDefinition()).getOffset() < nd.getOffset() )
nd = (ASTNode) cpp.getDefinition();
else
}
if( nd == null )
return true;
} else if( obj instanceof ASTNode ){
nd = (ASTNode) obj;