[clang-tools-extra] r188374 - cpp11-migrate: Fix silly logic error preventing multiple transforms

Edwin Vane edwin.vane at intel.com
Wed Aug 14 08:09:44 PDT 2013


Author: revane
Date: Wed Aug 14 10:09:44 2013
New Revision: 188374

URL: http://llvm.org/viewvc/llvm-project?rev=188374&view=rev
Log:
cpp11-migrate: Fix silly logic error preventing multiple transforms

A missed clause in an error test added in r188371 caused any use of the
migrator requesting multiple transforms to fail.


Modified:
    clang-tools-extra/trunk/cpp11-migrate/tool/Cpp11Migrate.cpp

Modified: clang-tools-extra/trunk/cpp11-migrate/tool/Cpp11Migrate.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/cpp11-migrate/tool/Cpp11Migrate.cpp?rev=188374&r1=188373&r2=188374&view=diff
==============================================================================
--- clang-tools-extra/trunk/cpp11-migrate/tool/Cpp11Migrate.cpp (original)
+++ clang-tools-extra/trunk/cpp11-migrate/tool/Cpp11Migrate.cpp Wed Aug 14 10:09:44 2013
@@ -264,7 +264,8 @@ int main(int argc, const char **argv) {
     return 1;
   }
 
-  if (std::distance(TransformManager.begin(), TransformManager.end()) > 1) {
+  if (std::distance(TransformManager.begin(), TransformManager.end()) > 1 &&
+      YAMLOnly) {
     llvm::errs() << "Header change description files requested for multiple "
                     "transforms.\nChanges from only one transform can be "
                     "recorded in a change description file.\n";





More information about the cfe-commits mailing list