Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[cdt-patch] FIXED 84749 - [Parser2] C++ : isConst not set on function declarator


FIXED 84749 - [Parser2] C++ :  isConst not set on function declarator

Devin Steffler
IBM's Eclipse CDT
Ottawa (Palladium), Ontario, Canada


Index: parser/org/eclipse/cdt/core/parser/tests/ast2/AST2CPPTests.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt-core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2CPPTests.java,v
retrieving revision 1.36
diff -u -r1.36 AST2CPPTests.java
--- parser/org/eclipse/cdt/core/parser/tests/ast2/AST2CPPTests.java	15 Feb 2005 19:49:53 -0000	1.36
+++ parser/org/eclipse/cdt/core/parser/tests/ast2/AST2CPPTests.java	16 Feb 2005 18:19:20 -0000
@@ -1339,11 +1339,9 @@
                 .getExpression();
         type = CPPVisitor.getExpressionType(ue);
 
-        //when 84749 is fixed, remove this assert and uncomment below.
-        assertSame(type, A);
-        //   		assertTrue( type instanceof IQualifierType );
-        //   		assertSame( ((IQualifierType) type).getType(), A );
-        //   		assertTrue( ((IQualifierType) type).isConst() );
+   		assertTrue( type instanceof IQualifierType );
+   		assertSame( ((IQualifierType) type).getType(), A );
+   		assertTrue( ((IQualifierType) type).isConst() );
     }
 
     public void testBug84710() throws Exception {
Index: parser/org/eclipse/cdt/internal/core/dom/parser/cpp/GNUCPPSourceParser.java
===================================================================
RCS file: /home/tools/org.eclipse.cdt-core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/GNUCPPSourceParser.java,v
retrieving revision 1.31
diff -u -r1.31 GNUCPPSourceParser.java
--- parser/org/eclipse/cdt/internal/core/dom/parser/cpp/GNUCPPSourceParser.java	15 Feb 2005 19:18:21 -0000	1.31
+++ parser/org/eclipse/cdt/internal/core/dom/parser/cpp/GNUCPPSourceParser.java	16 Feb 2005 18:18:59 -0000
@@ -3705,7 +3705,7 @@
                         isPureVirtual = true;
                      }
                   }
-                  if (afterCVModifier != LA(1) || LT(1) == IToken.tSEMI) {
+                  if (afterCVModifier != LA(1) || LT(1) == IToken.tSEMI || LT(1) == IToken.tLBRACE) {
                      // There were C++-specific clauses after
                      // const/volatile modifier
                      // Then it is a marker for the method

Back to the top