[libcxx-commits] [PATCH] D140117: [libc++][format] Renames __null_sentinel.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/include/__format/formatter_output.h:376
 
-struct __null_sentinel {};
+struct __nul_terminator {};
 
----------------
ldionne wrote:
> I think this was a typo? If so, can you please fix in a NFC?
`nul` instead of `null` was intentionally. It looks for the NUL character, so actually the older name was "wrong".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140117