[PATCH] -include option and -exclude ignore arguments finishing with '/'
Edwin Vane
edwin.vane at intel.com
Tue Jul 30 05:44:07 PDT 2013
Didn't we discuss a precondition that paths provided to `isFileIncluded()` must not have relative operators in them?
================
Comment at: unittests/cpp11-migrate/IncludeExcludeTest.cpp:66
@@ +65,3 @@
+ EXPECT_TRUE(IEManager.isFileIncluded("c/c2/c3/f.cpp"));
+}
+
----------------
Could we have some tests that have '.' in the path? I'd at least like to see a path like `/path/to/dir/.` work.
http://llvm-reviews.chandlerc.com/D1134
More information about the cfe-commits
mailing list