[flashrom-gerrit] Change in flashrom[master]: layout: Introduce layout_next()

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan, Daniel Campello, Anastasia Klimchuk, Peter Marheine.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/33542 )

Change subject: layout: Introduce layout_next()
......................................................................


Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/33542/comment/a1c1f9c8_c6ef867b 
PS3, Line 7: layout: Introduce layout_next()
           : 
           : Also, a `layout.c` internal version _layout_next()
> Maybe I am missing something, but this sounds like there are two functions: layout_next() and _layou […]
> I added both, the API one uses pointers to const struct romentries
> and the internal one pointers to mutable struct romentries.

I could just rename the latter to `mutable_layout_next()`, what
do you think?


File layout.c:

https://review.coreboot.org/c/flashrom/+/33542/comment/2f35d876_5c9d1a66 
PS3, Line 44: static struct romentry *_layout_next(
> Looks like this is declared without the leading underscore and used both ways.
> Their signatures differ, see comment thread on the commit message.