[libcxx-commits] [PATCH] D110598: [libc++] P0980R1 (constexpr std::string)

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/test/std/strings/basic.string/string.access/at.pass.cpp:58
+  {
+    typedef std::basic_string<char, std::char_traits<char>, min_allocator<char> > S;
     test(S(), 0);
----------------
Quuxplusone wrote:
> This whitespace change is gratuitous; notice that we're inside a `TEST_STD_VER >= 11` block here. (Generally speaking, please don't listen to clang-format.)
I didn't even notice that clang-format changed that. But since it doesn't hurt anything I would just leave it there, so you can just run clang-format and don't worry about changing some random part of the file (which is exactly what happened). Otherwise, what would even be the point of having a clang-format file at all?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110598