[libcxx-commits] [PATCH] D138307: [libc++] Remove default definition of std::char_traits

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

In D138307#3941791 <https://reviews.llvm.org/D138307#3941791>, @tahonermann wrote:

>> Also, would you have an appetite for reviving https://wg21.link/n1985?
>
> Not really. I could bring it to SG16, but I don't expect it to garner much favor. It might get some points for standardizing existing practice, but I expect the SG16 response to be that we don't want to encourage use of such extensions.

In that case, how about this:

1. We rip out `char_traits` support for `int` and other nonsensical stuff
2. We keep supporting it for `unsigned char` and `signed char`, but we make it deprecated
3. We remove it in 1-2 releases

If there's no desire to standardize it because the Committee thinks it's not a good idea, we should get clients to move away from it. We don't want to make it painful just for the sake of it, but we should initiate the movement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138307