[libcxx-commits] [PATCH] D142184: [libcxx] Add new line after assert message

[Vitaly Buka via Phabricator via libcxx-commits]

vitalybuka added inline comments.


================
Comment at: libcxx/src/verbose_abort.cpp:39
     std::vfprintf(stderr, format, list);
+    std::fputc('\n', stderr);
     va_end(list);
----------------
Mordante wrote:
> would it be possible to add require the newline in the `format` of `__libcpp_verbose_abort`? If not I'm happy with this approach.
> would it be possible to add require the newline in the `format` of `__libcpp_verbose_abort`? If not I'm happy with this approach.

Here? https://github.com/llvm/llvm-project/blob/de6b75c4ab130a3c798f691923b4b78b773e53cc/libcxx/include/__assert#L44

\n is reasonable for stderr, but I am not sure if \n is appropriate for Apple and Android stuff below


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142184