commit dcdff46dd8e6d749283fe6c43adfcfe780c1d562
author Eli Friedman <eli.friedman@gmail.com> Wed Aug 08 23:53:27 2012 +0000
committer Eli Friedman <eli.friedman@gmail.com> Wed Aug 08 23:53:27 2012 +0000
tree 3438dc34e0d7a8013238748fe8484fbf3b01f0cb
parent 8c6cb462946b8be19167fad7ff21d4f9fbaebfef

Minor simplification for r161534.



git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@161544 91177308-0d34-0410-b5e6-96231b3b80d8

include/clang/Parse/Parser.h

diff --git a/include/clang/Parse/Parser.h b/include/clang/Parse/Parser.h
index f02abc6..5faac1b 100644
--- a/include/clang/Parse/Parser.h
+++ b/include/clang/Parse/Parser.h
@@ -1560,7 +1560,7 @@
                             Decl *TagDecl);
 
   struct FieldCallback {
-    virtual Decl *invoke(ParsingFieldDeclarator &Field) = 0;
+    virtual void invoke(ParsingFieldDeclarator &Field) = 0;
     virtual ~FieldCallback() {}
 
   private:

lib/Parse/ParseDecl.cpp

diff --git a/lib/Parse/ParseDecl.cpp b/lib/Parse/ParseDecl.cpp
index a1e17a8..9f8f3ce 100644
--- a/lib/Parse/ParseDecl.cpp
+++ b/lib/Parse/ParseDecl.cpp
@@ -2891,14 +2891,13 @@
                        SmallVectorImpl<Decl *> &FieldDecls) :
           P(P), TagDecl(TagDecl), FieldDecls(FieldDecls) {}
 
-        virtual Decl *invoke(ParsingFieldDeclarator &FD) {
+        void invoke(ParsingFieldDeclarator &FD) {
           // Install the declarator into the current TagDecl.
           Decl *Field = P.Actions.ActOnField(P.getCurScope(), TagDecl,
                               FD.D.getDeclSpec().getSourceRange().getBegin(),
                                                  FD.D, FD.BitfieldSize);
           FieldDecls.push_back(Field);
           FD.complete(Field);
-          return Field;
         }
       } Callback(*this, TagDecl, FieldDecls);
 

lib/Parse/ParseObjc.cpp

diff --git a/lib/Parse/ParseObjc.cpp b/lib/Parse/ParseObjc.cpp
index e401983..b0aab88 100644
--- a/lib/Parse/ParseObjc.cpp
+++ b/lib/Parse/ParseObjc.cpp
@@ -308,16 +308,16 @@
     MethodImplKind(MethodImplKind) {
   }
 
-  Decl *invoke(ParsingFieldDeclarator &FD) {
+  void invoke(ParsingFieldDeclarator &FD) {
     if (FD.D.getIdentifier() == 0) {
       P.Diag(AtLoc, diag::err_objc_property_requires_field_name)
         << FD.D.getSourceRange();
-      return 0;
+      return;
     }
     if (FD.BitfieldSize) {
       P.Diag(AtLoc, diag::err_objc_property_bitfield)
         << FD.D.getSourceRange();
-      return 0;
+      return;
     }
 
     // Install the property declarator into interfaceDecl.
@@ -345,7 +345,6 @@
       Props.push_back(Property);
 
     FD.complete(Property);
-    return Property;
   }
 };
 
@@ -1307,7 +1306,7 @@
         P(P), IDecl(IDecl), visibility(V), AllIvarDecls(AllIvarDecls) {
       }
 
-      Decl *invoke(ParsingFieldDeclarator &FD) {
+      void invoke(ParsingFieldDeclarator &FD) {
         P.Actions.ActOnObjCContainerStartDefinition(IDecl);
         // Install the declarator into the interface decl.
         Decl *Field
@@ -1318,7 +1317,6 @@
         if (Field)
           AllIvarDecls.push_back(Field);
         FD.complete(Field);
-        return Field;
       }
     } Callback(*this, interfaceDecl, visibility, AllIvarDecls);