[PATCH] D42225: libcxx: Provide overloads for basic_filebuf::open() et al that take wchar_t* filenames on Windows.

Louis Dionne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 24 07:21:46 PDT 2021


ldionne added subscribers: mstorsjo, ldionne.
ldionne added a comment.
Herald added a subscriber: libcxx-commits.

@pcc @mstorsjo Are we aware of anyone using these extensions?

I would like to suggest that we either remove this extension if it's not useful, or make it unconditional (not only on Windows) if we really think it's that useful (but I'd like that to come with at least a paper proposing to add them to the standard). Carrying around an extension that's only enabled on one platform (and not the most widely used platform for libc++ at that) is kind of awkward.

WDYT?


Repository:
  rCXX libc++

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

https://reviews.llvm.org/D42225



More information about the cfe-commits mailing list