[libcxx-commits] [PATCH] D89674: [libcxx] [test] Fix all remaining issues with fs::path::string_type being wstring
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Oct 19 05:43:40 PDT 2020
ldionne added a comment.
In D89674#2338529 <https://reviews.llvm.org/D89674#2338529>, @mstorsjo wrote:
> In D89674#2338516 <https://reviews.llvm.org/D89674#2338516>, @ldionne wrote:
>
>> Accepting as long as you extract `PutEnv` into `filesystem_test_helper.h` and use it everywhere in the test suite!
>
> Sure, that sounds like a good idea, but I'd rather do that in a separate patch (superseding the one you commented on) and keeping this one as such (as a pure path/string handling fix).
Sounds good -- can you open the patch now? I want to make sure we don't accidentally forget to cleanup afterwards :-)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89674/new/
https://reviews.llvm.org/D89674
More information about the libcxx-commits
mailing list