[clang-tools-extra] r252207 - Refactor: Simplify boolean conditional return statements in clang-apply-replacements

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 5 12:59:18 PST 2015


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,




More information about the cfe-commits mailing list