Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47107 )
Change subject: option.h: Add strongly-typed `get_option` wrappers ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47107/2/src/include/option.h File src/include/option.h:
https://review.coreboot.org/c/coreboot/+/47107/2/src/include/option.h@34 PS2, Line 34: read_option_
I remember I mentioned `read_option()`. But in this context I would leave […]
I purposefully kept "read_option" unbroken in the function names to allow grepping for these. I can rename them, but I forgot what I wanted the greppability for.
https://review.coreboot.org/c/coreboot/+/47107/2/src/include/option.h@36 PS2, Line 36: type value; \
For the moment it seems a good idea to initialize it to 0. At least on […]
I can do it.