diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2012-07-03 00:15:11 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2012-07-03 00:15:11 +0000 |
| commit | 2e2c7a40b7164e8499b2551c363550d6da68dc97 (patch) | |
| tree | 56d8918f63c1b296163da87c3ca13694094cf30b /lib/Sema/SemaDeclObjC.cpp | |
| parent | e0c1168ec7910a1a7ed08df4d4f0c58c2fa2ecd1 (diff) | |
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....
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@159631 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaDeclObjC.cpp')
| -rw-r--r-- | lib/Sema/SemaDeclObjC.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Sema/SemaDeclObjC.cpp b/lib/Sema/SemaDeclObjC.cpp index 7342128715..87acd1e63d 100644 --- a/lib/Sema/SemaDeclObjC.cpp +++ b/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; |