[PATCH] D46887: Fix llvm::sys::path::remove_dots() to return "." instead of an empty path.

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 15 09:34:17 PDT 2018


clayborg added inline comments.


================
Comment at: unittests/Support/Path.cpp:1166
             remove_dots("././/foolz/wat", false, path::Style::posix));
-  EXPECT_EQ("", remove_dots("./////", false, path::Style::posix));
+  EXPECT_EQ(".", remove_dots("./////", false, path::Style::posix));
 
----------------
labath wrote:
> Another interesting test here would be to make sure this does return an empty path when the input is also empty.
I will add one.


Repository:
  rL LLVM

https://reviews.llvm.org/D46887





More information about the llvm-commits mailing list