Elyes Haouas has posted comments on this change by Elyes Haouas. ( https://review.coreboot.org/c/coreboot/+/83860?usp=email )
Change subject: i945: Use nullptr instead of NULL ......................................................................
Patch Set 9:
(1 comment)
Patchset:
PS9:
We decided on the leadership meeting and on CB:83459 that we won't change existing code for now. […]
I understand the decision made in the leadership meeting and on CB:83459 regarding not changing existing code at this time. However, I’d like to highlight that, eventually, we will need to move away from the NULL macro in favor of nullptr as we transition to C23.
The changes I made in nb/i945 code were intended as a pilot to demonstrate the use of nullptr in preparation for this future shift. Once C23 is fully adopted, it will be necessary to update our codebase accordingly. I believe that prohibiting developers from using new features introduced in C23 is not a sustainable approach, especially when planning for long-term compatibility.
Let’s consider gradually incorporating these updates, starting with specific areas, to better prepare for the transition to C23.