[libcxx-commits] [PATCH] D98097: [libc++] "Merged wording" for D98077 and D96986

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone marked an inline comment as done.
Quuxplusone added inline comments.


================
Comment at: libcxx/src/filesystem/filesystem_common.h:88
+format_string(const char* msg, ...) {
+  std::string ret;
+  va_list ap;
----------------
Mordante wrote:
> For consistency can you remove `std::`?
Done (and also on line 71). This file does contain some `std::`s, on `declval` and `array`, but not on `string`. (Personally I wish it would put `std::` on everything, just like user code should. We're not doing anything "special" in this file that //requires// us to deviate from the usual good practice; the current use of unqualified `string` is just inessential mental noise IMO.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98097