Remove a redundant assignment to the FDecl variable from *inside* it's

initializer.

I really feel like Clang should warn about this, but I can't describe
a good reason. GCC will warn on this in some cases under
-Wsequence-point, but it actually seems like a false positive for that
warning....

llvm-svn: 159631

1 files changed, 1 insertions, 1 deletions

diff --git a/clang/lib/Sema/SemaDeclObjC.cpp b/clang/lib/Sema/SemaDeclObjC.cpp
index 7342128715b..87acd1e63d0 100644
--- a/clang/lib/Sema/SemaDeclObjC.cpp
+++ b/clang/lib/Sema/SemaDeclObjC.cpp
@@ -273,7 +273,7 @@ void Sema::ActOnStartOfObjCMethodOrCFunctionDef(Scope *FnBodyScope, Decl *D,
   assert((getCurMethodDecl() == 0 && getCurFunctionDecl() == 0) &&
          "Method/c-function parsing confused");
   if (!parseMethod) {
-    FunctionDecl *FDecl = FDecl = dyn_cast_or_null<FunctionDecl>(D);
+    FunctionDecl *FDecl = dyn_cast_or_null<FunctionDecl>(D);
     // If we don't have a valid c-function decl, simply return.
     if (!FDecl)
       return;