[PATCH] -include option and -exclude ignore arguments finishing with '/'

Ariel Bernal ariel.j.bernal at intel.com
Fri Jul 12 09:54:08 PDT 2013


Hi Sarcasm,

When transforming headers using -include, arguments that ended with '/' were ignored.

http://llvm-reviews.chandlerc.com/D1134

Files:
  cpp11-migrate/Core/IncludeExcludeInfo.cpp

Index: cpp11-migrate/Core/IncludeExcludeInfo.cpp
===================================================================
--- cpp11-migrate/Core/IncludeExcludeInfo.cpp
+++ cpp11-migrate/Core/IncludeExcludeInfo.cpp
@@ -66,10 +66,12 @@
     // off or it interferes with prefix checking.
     if (Path.endswith("/."))
       List.push_back(Path.slice(0, Path.size() - 2).str());
+    else if (Path.endswith("/"))
+      List.push_back(Path.slice(0, Path.size() - 1).str());
     else
       List.push_back(Path.str());
 
-    llvm::errs() << "Parse: " <<List.back() << "\n";
+    llvm::errs() << "Parse: " << List.back() << "\n";
   }
   return error_code::success();
 }
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1134.1.patch
Type: text/x-patch
Size: 663 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130712/1f3cca51/attachment.bin>


More information about the cfe-commits mailing list