[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