Attention is currently required from: Anastasia Klimchuk.
Patch set 1:Code-Review +2
View Change
1 comment:
File layout.c:
Patch Set #1, Line 117: bool
Hmmm, returning an error code would've involved changing less things, but it's not a big deal. We don't seem to have any rules to prefer returning error codes (success is 0) or boolean success/failure (success is not-0, aka true), but the former style seems more common in flashrom.
No changes needed, just sharing our thoughts.
To view, visit change 70539. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iba2971846938fe95412f0a69ff3c069ee2d049b6
Gerrit-Change-Number: 70539
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Comment-Date: Fri, 09 Dec 2022 12:30:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment