Author: alexfh
Date: Thu Nov  5 14:59:17 2015
New Revision: 252207

URL: http://llvm.org/viewvc/llvm-project?rev=252207&view=rev
Log:
Refactor: Simplify boolean conditional return statements in 
clang-apply-replacements

Differential revision: http://reviews.llvm.org/D10025

Patch by Richard Thomson!

Modified:
    
clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp

Modified: 
clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
URL: 
http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp?rev=252207&r1=252206&r2=252207&view=diff
==============================================================================
--- 
clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
 (original)
+++ 
clang-tools-extra/trunk/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
 Thu Nov  5 14:59:17 2015
@@ -184,10 +184,7 @@ bool mergeAndDeduplicate(const TUReplace
   }
 
   // Ask clang to deduplicate and report conflicts.
-  if (deduplicateAndDetectConflicts(GroupedReplacements, SM))
-    return false;
-
-  return true;
+  return !deduplicateAndDetectConflicts(GroupedReplacements, SM);
 }
 
 bool applyReplacements(const FileToReplacementsMap &GroupedReplacements,


_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to