4 comments:
File src/commonlib/bsd/cbfs_private.c:
Patch Set #8, Line 8: size_t devsize
Maybe constify this?
Patch Set #8, Line 30: bool do_hash
Maybe constify this?
Patch Set #8, Line 44: while ((ret_header = read_next_header(dev, &offset, &mdata.h)) == CB_SUCCESS) {
Who knows what it does... […]
In the case above, fixing it would result in two nested if's, and it would bring the line length past 96 characters. So I wouldn't touch it 😄
I think checkpatch only complains on instances inside if statements. It does not seem to complain on loops, maybe because it does makes sense to use such a construct for loops.
File src/commonlib/bsd/include/commonlib/bsd/cbfs_serialized.h:
Patch Set #8, Line 127: /* Actual size in CBFS may be larger/smaller than struct size! */
Silly question: what does this mean?
To view, visit change 38421. To unsubscribe, or for help writing mail filters, visit settings.