[coreboot-gerrit] Change in ...coreboot[master]: src: Use 'include <stdlib.h>' when appropriate

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32023 )

Change subject: src: Use 'include <stdlib.h>' when appropriate
......................................................................

Patch Set 12:

(2 comments)

https://review.coreboot.org/#/c/32023/12//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32023/12//COMMIT_MSG@12 
PS12, Line 12:  include <commonlib/stdlib.h>
Looking at the stat, this change seems to be more
about removing #includes. And I wonder what rules
were followed to do that? I know it's a huge effort
and there were probably already hours spent to
review this; but what do we gain from it? This
change seems to clean up some cases of indirect
includes while it creates some other?

If there is no automatic way to decide this, will
the cleanup ever end? or will we clean up our
cleanups forever in a loop?

https://review.coreboot.org/#/c/32023/12/src/arch/x86/include/arch/acpigen.h 
File src/arch/x86/include/arch/acpigen.h:

https://review.coreboot.org/#/c/32023/12/src/arch/x86/include/arch/acpigen.h... 
PS12, Line 168: size_t
what about this?