[PATCH] D59079: [clang][Tooling] Delete dots and dotdots when generating absolute paths

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 7 02:01:43 PST 2019


hokein accepted this revision.
hokein added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Tooling/JSONCompilationDatabase.cpp:373
       llvm::sys::path::append(AbsolutePath, FileName);
+      llvm::sys::path::remove_dots(AbsolutePath, true);
       llvm::sys::path::native(AbsolutePath, NativeFilePath);
----------------
nit: add the `/*remove_dot_dot=*/` for the `true` parameter.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59079/new/

https://reviews.llvm.org/D59079





More information about the cfe-commits mailing list