[PATCH] -include option and -exclude ignore arguments finishing with '/'
Edwin Vane
edwin.vane at intel.com
Tue Jul 30 10:22:16 PDT 2013
================
Comment at: unittests/cpp11-migrate/IncludeExcludeTest.cpp:58
@@ +57,3 @@
+ llvm::error_code Err = IEManager.readListFromString(
+ /*include=*/ "a/.,b/b2/,c/c2/c3/../../c2/,b/b2/./b3/,/b/b2/.",
+ /*exclude=*/ "");
----------------
I think this test would be better if you changed c/c2/c3/../../c4/. This way you can ensure that files from c/c2 are **not** included in the tests below.
Also, your use of b/b2/./b3/ is covered by /b/b2/. Can you use a different directory structure between these two to make sure there is no overlap?
http://llvm-reviews.chandlerc.com/D1134
More information about the cfe-commits
mailing list