[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
[cdt-patch] Applied [HEAD] - IASTElaboratedTypeSpecifier should inherit from IASTOffsetableNamedElement
|
CORE
Removed warnings from SymbolTable & QuickParseCallback (removing
implicit accessor generation).
Made IASTElaboratedTypeSpecifier derive from
IASTOffsetableNamedElement (as it should).
TESTS
Removed warnings from AutomatedIntegrationSuite.java (removing
implicit accessor generation).
JohnC
Index: ChangeLog
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core.tests/ChangeLog,v
retrieving revision 1.61
diff -u -r1.61 ChangeLog
--- ChangeLog 14 Aug 2003 15:33:32 -0000 1.61
+++ ChangeLog 14 Aug 2003 19:49:29 -0000
@@ -1,4 +1,7 @@
2003-08-14 John Camelon
+ Removed warnings from AutomatedIntegrationSuite.java (removing implicit accessor generation).
+
+2003-08-14 John Camelon
Added CompleteParseASTTest::testArrayModExpression(), testPointerVariable() &
testExceptionSpecification().
Index: suite/org/eclipse/cdt/core/suite/AutomatedIntegrationSuite.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core.tests/suite/org/eclipse/cdt/core/suite/AutomatedIntegrationSuite.java,v
retrieving revision 1.13
diff -u -r1.13 AutomatedIntegrationSuite.java
--- suite/org/eclipse/cdt/core/suite/AutomatedIntegrationSuite.java 13 Aug 2003 17:45:21 -0000 1.13
+++ suite/org/eclipse/cdt/core/suite/AutomatedIntegrationSuite.java 14 Aug 2003 19:49:30 -0000
@@ -55,8 +55,8 @@
private int numberOfFailedTests = 0;
private int numberOfFailedFailedTests = 0;
// switching to failed tests
- private boolean failedTests = false;
- private boolean skipTest = false;
+ protected boolean failedTests = false;
+ protected boolean skipTest = false;
public AutomatedIntegrationSuite() {}
@@ -220,14 +220,14 @@
System.out.println();
}
- private void startSuccessTests() {
+ protected void startSuccessTests() {
failedTests = false;
System.out.println();
System.out.println("*** Starting success tests ***");
System.out.println();
}
- private void startFailedTests() {
+ protected void startFailedTests() {
failedTests = true;
System.out.println();
System.out.println("*** Starting failed tests ***");
Index: parser/ChangeLog
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/ChangeLog,v
retrieving revision 1.101
diff -u -r1.101 ChangeLog
--- parser/ChangeLog 14 Aug 2003 15:33:27 -0000 1.101
+++ parser/ChangeLog 14 Aug 2003 19:48:45 -0000
@@ -1,4 +1,8 @@
2003-08-14 John Camelon
+ Removed warnings from SymbolTable & QuickParseCallback (removing implicit accessor generation).
+ Made IASTElaboratedTypeSpecifier derive from IASTOffsetableNamedElement (as it should).
+
+2003-08-14 John Camelon
Added X-Reference support for ArrayModifiers and Exception Specifications.
Fixed Bug 41551 - HandleInclusion always throws ScannerException on local includes.
Index: parser/org/eclipse/cdt/core/parser/ast/IASTElaboratedTypeSpecifier.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/parser/ast/IASTElaboratedTypeSpecifier.java,v
retrieving revision 1.7
diff -u -r1.7 IASTElaboratedTypeSpecifier.java
--- parser/org/eclipse/cdt/core/parser/ast/IASTElaboratedTypeSpecifier.java 12 Aug 2003 20:40:04 -0000 1.7
+++ parser/org/eclipse/cdt/core/parser/ast/IASTElaboratedTypeSpecifier.java 14 Aug 2003 19:48:45 -0000
@@ -18,7 +18,7 @@
* @author jcamelon
*
*/
-public interface IASTElaboratedTypeSpecifier extends IASTTypeSpecifier, IASTOffsetableElement, IASTQualifiedNameElement, ISourceElementCallbackDelegate {
+public interface IASTElaboratedTypeSpecifier extends IASTTypeSpecifier, IASTOffsetableElement, IASTQualifiedNameElement, ISourceElementCallbackDelegate, IASTOffsetableNamedElement {
public String getName();
public ASTClassKind getClassKind();
Index: parser/org/eclipse/cdt/internal/core/parser/QuickParseCallback.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/QuickParseCallback.java,v
retrieving revision 1.3
diff -u -r1.3 QuickParseCallback.java
--- parser/org/eclipse/cdt/internal/core/parser/QuickParseCallback.java 22 Jul 2003 18:32:40 -0000 1.3
+++ parser/org/eclipse/cdt/internal/core/parser/QuickParseCallback.java 14 Aug 2003 19:48:46 -0000
@@ -27,9 +27,9 @@
public class QuickParseCallback extends NullSourceElementRequestor implements IQuickParseCallback
{
- private IASTCompilationUnit compilationUnit = null;
- private List inclusions = new ArrayList();
- private List macros = new ArrayList();
+ protected IASTCompilationUnit compilationUnit = null;
+ protected List inclusions = new ArrayList();
+ protected List macros = new ArrayList();
public Iterator getInclusions()
{
Index: parser/org/eclipse/cdt/internal/core/parser/TokenDuple.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/TokenDuple.java,v
retrieving revision 1.9
diff -u -r1.9 TokenDuple.java
--- parser/org/eclipse/cdt/internal/core/parser/TokenDuple.java 12 Aug 2003 18:19:38 -0000 1.9
+++ parser/org/eclipse/cdt/internal/core/parser/TokenDuple.java 14 Aug 2003 19:48:46 -0000
@@ -27,7 +27,7 @@
firstToken = first;
lastToken = last;
}
- private final IToken firstToken, lastToken;
+ protected final IToken firstToken, lastToken;
/**
* @return
*/
Index: parser/org/eclipse/cdt/internal/core/parser/ast/complete/ASTElaboratedTypeSpecifier.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/ast/complete/ASTElaboratedTypeSpecifier.java,v
retrieving revision 1.3
diff -u -r1.3 ASTElaboratedTypeSpecifier.java
--- parser/org/eclipse/cdt/internal/core/parser/ast/complete/ASTElaboratedTypeSpecifier.java 12 Aug 2003 20:40:04 -0000 1.3
+++ parser/org/eclipse/cdt/internal/core/parser/ast/complete/ASTElaboratedTypeSpecifier.java 14 Aug 2003 19:48:46 -0000
@@ -17,7 +17,7 @@
import org.eclipse.cdt.core.parser.ast.ASTNotImplementedException;
import org.eclipse.cdt.core.parser.ast.IASTElaboratedTypeSpecifier;
import org.eclipse.cdt.internal.core.parser.ast.ASTQualifiedNamedElement;
-import org.eclipse.cdt.internal.core.parser.ast.Offsets;
+import org.eclipse.cdt.internal.core.parser.ast.NamedOffsets;
import org.eclipse.cdt.internal.core.parser.pst.ISymbol;
/**
@@ -29,7 +29,7 @@
private final boolean isForwardDeclaration;
private final ASTClassKind kind;
private final ASTQualifiedNamedElement qualifiedName;
- private Offsets offsets = new Offsets();
+ private NamedOffsets offsets = new NamedOffsets();
private final ASTReferenceStore store;
/**
@@ -38,11 +38,12 @@
* @param startingOffset
* @param endOffset
*/
- public ASTElaboratedTypeSpecifier(ISymbol checkSymbol, ASTClassKind kind, int startingOffset, int endOffset, List references, boolean isDecl )
+ public ASTElaboratedTypeSpecifier(ISymbol checkSymbol, ASTClassKind kind, int startingOffset, int nameOffset, int endOffset, List references, boolean isDecl )
{
super( checkSymbol );
this.kind = kind;
setStartingOffset( startingOffset );
+ setNameOffset( nameOffset );
setEndingOffset( endOffset );
qualifiedName = new ASTQualifiedNamedElement( getOwnerScope(), checkSymbol.getName() );
store = new ASTReferenceStore( references );
@@ -126,4 +127,16 @@
{
return qualifiedName.getFullyQualifiedName();
}
+ /* (non-Javadoc)
+ * @see org.eclipse.cdt.core.parser.ast.IASTOffsetableNamedElement#getNameOffset()
+ */
+ public int getNameOffset() {
+ return offsets.getNameOffset();
+ }
+ /* (non-Javadoc)
+ * @see org.eclipse.cdt.core.parser.ast.IASTOffsetableNamedElement#setNameOffset(int)
+ */
+ public void setNameOffset(int o) {
+ offsets.setNameOffset(o);
+ }
}
Index: parser/org/eclipse/cdt/internal/core/parser/ast/complete/CompleteParseASTFactory.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/ast/complete/CompleteParseASTFactory.java,v
retrieving revision 1.16
diff -u -r1.16 CompleteParseASTFactory.java
--- parser/org/eclipse/cdt/internal/core/parser/ast/complete/CompleteParseASTFactory.java 14 Aug 2003 15:33:27 -0000 1.16
+++ parser/org/eclipse/cdt/internal/core/parser/ast/complete/CompleteParseASTFactory.java 14 Aug 2003 19:48:50 -0000
@@ -1344,7 +1344,6 @@
{
if( checkSymbol == null )
{
-
checkSymbol = pst.newDerivableContainerSymbol( lastToken.getImage(), pstType );
checkSymbol.setIsForwardDeclaration( true );
try
@@ -1357,7 +1356,7 @@
}
ASTElaboratedTypeSpecifier elab =
- new ASTElaboratedTypeSpecifier( checkSymbol, kind, startingOffset, endOffset, references, isForewardDecl );
+ new ASTElaboratedTypeSpecifier( checkSymbol, kind, startingOffset, name.getFirstToken().getOffset(), endOffset, references, isForewardDecl );
try
{
Index: parser/org/eclipse/cdt/internal/core/parser/ast/quick/ASTElaboratedTypeSpecifier.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/ast/quick/ASTElaboratedTypeSpecifier.java,v
retrieving revision 1.6
diff -u -r1.6 ASTElaboratedTypeSpecifier.java
--- parser/org/eclipse/cdt/internal/core/parser/ast/quick/ASTElaboratedTypeSpecifier.java 12 Aug 2003 20:40:04 -0000 1.6
+++ parser/org/eclipse/cdt/internal/core/parser/ast/quick/ASTElaboratedTypeSpecifier.java 14 Aug 2003 19:48:50 -0000
@@ -16,7 +16,7 @@
import org.eclipse.cdt.core.parser.ast.IASTElaboratedTypeSpecifier;
import org.eclipse.cdt.core.parser.ast.IASTScope;
import org.eclipse.cdt.internal.core.parser.ast.ASTQualifiedNamedElement;
-import org.eclipse.cdt.internal.core.parser.ast.Offsets;
+import org.eclipse.cdt.internal.core.parser.ast.NamedOffsets;
/**
* @author jcamelon
@@ -25,7 +25,7 @@
public class ASTElaboratedTypeSpecifier implements IASTElaboratedTypeSpecifier
{
- private Offsets offsets = new Offsets();
+ private NamedOffsets offsets = new NamedOffsets();
private final String typeName;
private final ASTClassKind classKind;
private final ASTQualifiedNamedElement qualifiedName;
@@ -35,12 +35,13 @@
* @param startingOffset
* @param endOffset
*/
- public ASTElaboratedTypeSpecifier(IASTScope scope, ASTClassKind elaboratedClassKind, String typeName, int startingOffset, int endOffset)
+ public ASTElaboratedTypeSpecifier(IASTScope scope, ASTClassKind elaboratedClassKind, String typeName, int startingOffset, int nameOffset, int endOffset)
{
classKind = elaboratedClassKind;
this.typeName = typeName;
- offsets.setStartingOffset( startingOffset );
- offsets.setEndingOffset( endOffset );
+ setStartingOffset( startingOffset );
+ setNameOffset( nameOffset );
+ setEndingOffset( endOffset );
qualifiedName = new ASTQualifiedNamedElement( scope, typeName );
}
/* (non-Javadoc)
@@ -119,4 +120,16 @@
public void exitScope(ISourceElementRequestor requestor)
{
}
+ /* (non-Javadoc)
+ * @see org.eclipse.cdt.core.parser.ast.IASTOffsetableNamedElement#getNameOffset()
+ */
+ public int getNameOffset() {
+ return offsets.getNameOffset();
+ }
+ /* (non-Javadoc)
+ * @see org.eclipse.cdt.core.parser.ast.IASTOffsetableNamedElement#setNameOffset(int)
+ */
+ public void setNameOffset(int o) {
+ offsets.setNameOffset(o);
+ }
}
Index: parser/org/eclipse/cdt/internal/core/parser/ast/quick/QuickParseASTFactory.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/ast/quick/QuickParseASTFactory.java,v
retrieving revision 1.24
diff -u -r1.24 QuickParseASTFactory.java
--- parser/org/eclipse/cdt/internal/core/parser/ast/quick/QuickParseASTFactory.java 14 Aug 2003 15:33:27 -0000 1.24
+++ parser/org/eclipse/cdt/internal/core/parser/ast/quick/QuickParseASTFactory.java 14 Aug 2003 19:48:52 -0000
@@ -265,6 +265,6 @@
public IASTElaboratedTypeSpecifier createElaboratedTypeSpecifier(IASTScope scope, ASTClassKind elaboratedClassKind, ITokenDuple typeName, int startingOffset, int endOffset, boolean isForewardDecl)
{
- return new ASTElaboratedTypeSpecifier( scope, elaboratedClassKind, typeName.toString(), startingOffset, endOffset );
+ return new ASTElaboratedTypeSpecifier( scope, elaboratedClassKind, typeName.toString(), startingOffset, typeName.getFirstToken().getOffset(), endOffset );
}
}
Index: parser/org/eclipse/cdt/internal/core/parser/pst/ParserSymbolTable.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/pst/ParserSymbolTable.java,v
retrieving revision 1.12
diff -u -r1.12 ParserSymbolTable.java
--- parser/org/eclipse/cdt/internal/core/parser/pst/ParserSymbolTable.java 7 Aug 2003 14:46:58 -0000 1.12
+++ parser/org/eclipse/cdt/internal/core/parser/pst/ParserSymbolTable.java 14 Aug 2003 19:48:54 -0000
@@ -82,7 +82,7 @@
* @return Declaration
* @throws ParserSymbolTableException
*/
- static private void lookup( LookupData data, IContainerSymbol inSymbol ) throws ParserSymbolTableException
+ static protected void lookup( LookupData data, IContainerSymbol inSymbol ) throws ParserSymbolTableException
{
if( data.type != TypeInfo.t_any && data.type.compareTo(TypeInfo.t_class) < 0 && data.upperType.compareTo(TypeInfo.t_union) > 0 ){
throw new ParserSymbolTableException( ParserSymbolTableException.r_BadTypeInfo );
@@ -218,7 +218,7 @@
*
* Look for data.name in our collection _containedDeclarations
*/
- private static boolean lookupInContained( LookupData data, IContainerSymbol lookIn ){
+ protected static boolean lookupInContained( LookupData data, IContainerSymbol lookIn ){
boolean foundSomething = false;
ISymbol temp = null;
@@ -436,7 +436,7 @@
* 3.4-1 "Name lookup may associate more than one declaration with a name if
* it finds the name to be a function name"
*/
- private static boolean isValidOverload( ISymbol origSymbol, ISymbol newSymbol ){
+ protected static boolean isValidOverload( ISymbol origSymbol, ISymbol newSymbol ){
TypeInfo.eType origType = origSymbol.getType();
TypeInfo.eType newType = newSymbol.getType();
@@ -460,7 +460,7 @@
return false;
}
- private static boolean isValidOverload( LinkedList origList, ISymbol newSymbol ){
+ protected static boolean isValidOverload( LinkedList origList, ISymbol newSymbol ){
if( origList.size() == 1 ){
return isValidOverload( (ISymbol)origList.getFirst(), newSymbol );
} else if ( origList.size() > 1 ){
@@ -534,7 +534,7 @@
* all, when looking for functions with no parameters, an empty list must be
* provided in data.parameters.
*/
- static private ISymbol resolveAmbiguities( LookupData data ) throws ParserSymbolTableException{
+ static protected ISymbol resolveAmbiguities( LookupData data ) throws ParserSymbolTableException{
ISymbol decl = null;
ISymbol obj = null;
IContainerSymbol cls = null;
@@ -655,7 +655,7 @@
}
}
- static private IParameterizedSymbol resolveFunction( LookupData data, LinkedList functions ) throws ParserSymbolTableException{
+ static protected IParameterizedSymbol resolveFunction( LookupData data, LinkedList functions ) throws ParserSymbolTableException{
if( functions == null ){
return null;
}
@@ -983,7 +983,7 @@
return -1;
}
- static private void getAssociatedScopes( ISymbol symbol, HashSet associated ){
+ static protected void getAssociatedScopes( ISymbol symbol, HashSet associated ){
if( symbol == null ){
return;
}
@@ -1027,7 +1027,7 @@
}
}
- static private boolean okToAddUsingDeclaration( ISymbol obj, IContainerSymbol context ){
+ static protected boolean okToAddUsingDeclaration( ISymbol obj, IContainerSymbol context ){
boolean okToAdd = false;
//7.3.3-5 A using-declaration shall not name a template-id
@@ -1426,7 +1426,7 @@
* remaining TypeInfo's represent the object.
*/
// TODO move this to ITypeInfo ?
- static private TypeInfo getFlatTypeInfo( TypeInfo topInfo ){
+ static protected TypeInfo getFlatTypeInfo( TypeInfo topInfo ){
TypeInfo returnInfo = topInfo;
TypeInfo info = null;
@@ -1794,7 +1794,7 @@
private LinkedList undoList = new LinkedList();
private HashSet markSet = new HashSet();
- private void pushCommand( Command command ){
+ protected void pushCommand( Command command ){
undoList.addFirst( command );
}
@@ -3257,7 +3257,7 @@
}
private boolean isVirtual = false;
- private ISymbol parent = null;
+ protected ISymbol parent = null;
private final ASTAccessVisibility access;
private final int offset;
private final List references;
Index: parser/org/eclipse/cdt/internal/core/parser/pst/TypeInfo.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/parser/pst/TypeInfo.java,v
retrieving revision 1.3
diff -u -r1.3 TypeInfo.java
--- parser/org/eclipse/cdt/internal/core/parser/pst/TypeInfo.java 7 Aug 2003 14:46:58 -0000 1.3
+++ parser/org/eclipse/cdt/internal/core/parser/pst/TypeInfo.java 14 Aug 2003 19:48:54 -0000
@@ -114,7 +114,7 @@
//public static final eType t_templateParameter = new eType( 18 );
public static class eType implements Comparable{
- private eType( int v ){
+ protected eType( int v ){
_val = v;
}