[PATCH] -include option and -exclude ignore arguments finishing with '/'
Ariel Bernal
ariel.j.bernal at intel.com
Tue Jul 30 11:08:49 PDT 2013
================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.cpp:64
@@ +63,3 @@
+ if (PathT.empty()) {
+ llvm::errs() << "Unable to remove relative operators: " << Path << "\n";
+ return "";
----------------
Guillaume Papin wrote:
> Wouldn't be better to print this message in parseCLInput() and instead of printing `Path` print the user-provided path (aka. `*I`)?
sure no problem
================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.cpp:54
@@ -50,1 +53,3 @@
+/// \brief Helper function for removing relative operations from a given
+/// path i.e. "..", ".".
----------------
Guillaume Papin wrote:
> operations?
maybe operators
================
Comment at: cpp11-migrate/Core/IncludeExcludeInfo.cpp:59
@@ +58,3 @@
+ sys::path::const_iterator PathE = sys::path::end(Path);
+ SmallVector<StringRef, 32> PathT;
+ while (PathI != PathE) {
----------------
Guillaume Papin wrote:
> no big deal but using 32 levels of directories seems quite uncommon to me.
yes sure I can use 16.
http://llvm-reviews.chandlerc.com/D1134
More information about the cfe-commits
mailing list