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

Guillaume Papin guillaume.papin at epitech.eu
Fri Jul 12 10:12:15 PDT 2013


  Have you considered using things like:
  - `llvm::sys::fs::canonicalize()` (http://llvm.org/docs/doxygen/html/namespacellvm_1_1sys_1_1fs.html#ac81340f79222ebe6143d9e86257a09eb)
  - `llvm::sys::fs::equivalent()` (http://llvm.org/docs/doxygen/html/namespacellvm_1_1sys_1_1fs.html#ae27a8654454573d56ca58b060c1367b8)
  ?

  Because I'm afraid we can find easily find some similar bugs.

  And maybe adding some tests?

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



More information about the cfe-commits mailing list