[Lldb-commits] [PATCH] D46783: FileSpec objects that resolve to "." should have "." in m_filename and m_directory empty.

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue May 15 08:13:30 PDT 2018


clayborg added a comment.

In https://reviews.llvm.org/D46783#1097549, @labath wrote:

> Preserving the dot if it is the only path component is perfectly reasonable -- "" is not a valid path and we shouldn't convert a valid path into an invalid one. However, I think this should be done on the llvm side, in the `remote_dots` function and not as a workaround on top of it.


so the issue isn't just keeping a single ".", but "./" and "foo/.." and many others that result in the "." as being the result. And I don't think changing a function that is supposed to remove dots to not actually remove them is a good idea.


https://reviews.llvm.org/D46783





More information about the lldb-commits mailing list