Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/31247 )
Change subject: layout.c: Don't use global variables for included regions ......................................................................
Patch Set 6: Code-Review+2
(5 comments)
https://review.coreboot.org/#/c/31247/1/layout.h File layout.h:
https://review.coreboot.org/#/c/31247/1/layout.h@65 PS1, Line 65: const
same
Done
https://review.coreboot.org/#/c/31247/1/layout.h@65 PS1, Line 65: int process_include_args(const struct layout_include_args * const args, struct flashrom_layout *const l);
I would prefer to have the output parameter first.
Done
https://review.coreboot.org/#/c/31247/1/layout.c File layout.c:
https://review.coreboot.org/#/c/31247/1/layout.c@109 PS1, Line 109: while(tmp) {
missing space
Done
https://review.coreboot.org/#/c/31247/1/layout.c@117 PS1, Line 117: tmp = malloc(sizeof(struct layout_include_args));
null-check
Done
https://review.coreboot.org/#/c/31247/1/layout.c@144 PS1, Line 144:
spurious space between `*` and `const`
Done