Patch set 6:Code-Review +2

View Change

5 comments:

• File layout.h:

  • Patch Set #1, Line 65: const

    same

    Done

  • Patch Set #1, 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

• File layout.c:

  • Patch Set #1, Line 109: while(tmp) {

    missing space

    Done

  • Patch Set #1, Line 117: tmp = malloc(sizeof(struct layout_include_args));

    null-check

    Done

  • Patch Set #1, Line 144:

    spurious space between `*` and `const`

    Done

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