Patch set 2:Code-Review +1

View Change

2 comments:

• File layout.c:

  • Patch Set #2, Line 274:

    
    const struct romentry *layout_next_included(
    		const struct flashrom_layout *const layout, const struct romentry *iterator)
    {
    	while ((iterator = layout_next(layout, iterator))) {
    		if (iterator->included)
    			break;
    	}
    	return iterator;
    }
    

    Maybe move this above _layout_next to align with the header file

• File libflashrom.c:

  • Patch Set #2, Line 351:

    			chip_entry = layout_next(&chip_layout->base, chip_entry);
    			dump_entry = layout_next(&dump_layout.base, dump_entry);
    

    Looks weird that one of these uses '->' and the other uses '.'

To view, visit change 33542. To unsubscribe, or for help writing mail filters, visit settings.