[coreboot-gerrit] [XS] Change in coreboot[main]: stddef.h: Introduce nullptr constant

[Julius Werner (Code Review)]

Attention is currently required from: Elyes Haouas, Felix Singer, Jakub Czapiga, Martin L Roth, Maximilian Brune, Nico Huber.

Julius Werner has posted comments on this change by Elyes Haouas. ( https://review.coreboot.org/c/coreboot/+/83459?usp=email )

Change subject: stddef.h: Introduce nullptr constant
......................................................................

Patch Set 26: Code-Review+2

(1 comment)

File src/include/stddef.h:

https://review.coreboot.org/c/coreboot/+/83459/comment/2cb60a56_06552442?usp... :
PS26, Line 25: #define nullptr ((void *)0)
> I also don't think people are against using it. […]

I don't remember exactly what was said but I agree with Felix here: I think the important part is that we don't want to rewrite existing code and keep using NULL by default in new code. But I'm okay with adding these definitions here and resolving NULL to nullptr when available for whatever tiny type-checking advantage that may bring. Having the name nullptr defined may be useful in rare cases where we're importing code wholesale from somewhere that uses it, although I don't think we should start using it in normal coreboot code.