[PATCH] D43159: Modernize: Use nullptr more.

Dimitry Andric via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 11 14:40:33 PST 2018


dim added a comment.

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.


Repository:
  rCXX libc++

https://reviews.llvm.org/D43159





More information about the cfe-commits mailing list