[PATCH] D43159: Modernize: Use nullptr more.
Bruce Mitchener via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 12 22:53:59 PST 2018
brucem added a comment.
In https://reviews.llvm.org/D43159#1004639, @dim wrote:
> In https://reviews.llvm.org/D43159#1004625, @EricWF wrote:
>
> > So my main concern with this patch is that `nullptr` is actually `#defined`'ed in C++03 mode. That definition comes from the `__nullptr` header, and therefore we would need to add that header to each include which uses it. Which kind of sucks.
>
>
> Indeed, but isn't `nullptr` used in many headers already? And as far as I can see, none of those includes <__nullptr> explicitly, so the definition must come from some transitive include.
It is handled in the wrapper around `stddef.h` so it all should just work.
Repository:
rCXX libc++
https://reviews.llvm.org/D43159
More information about the cfe-commits
mailing list