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

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik accepted this revision.
philnik added a comment.
This revision is now accepted and ready to land.

LGTM with green CI.



================
Comment at: libcxx/test/std/strings/basic.string/string.modifiers/string_append/push_back.pass.cpp:27
+  template <>
+  struct char_traits<VeryLarge> {
+    using char_type           = VeryLarge;
----------------
I'd rather avoid opening the `std` namespace and instead use `template<> struct std::char_traits<VeryLarge>`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138307