[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