[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