Patch Set 1: Code-Review+1

(1 comment)

Hmmm, I remember some resistance towards these correct-by-definition
but unnecessary-given-we-know-the-compiler changes. I guess we should
have a broader discussion about it at some point.

Personally, I'm ok with it. Would prefer `1u`, though (lower case
to be more distinct from the number part, and the `L` doesn't seem
needed).

Thanks for reviewing. I updated the short description. I'll keep the code since both upper/lower-cases of "unsigned" symbols exists in the current coreboot code base. We can change it if we have an unify coding style.

Indeed we might need to discuss how are we process the errors produced by sanitizers( coreboot have two types: undefined behavior and address-specific. There are still a few compliants( not sure if its real bug now) from UBsan. I'll file the bug then.

Patch set 2:Code-Review +1

View Change

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6eeee1e92b616f68aa60eb74771912dd00ff7712
Gerrit-Change-Number: 49076
Gerrit-PatchSet: 2
Gerrit-Owner: Shawn C <citypw@hardenedlinux.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: Shawn C <citypw@hardenedlinux.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Tue, 05 Jan 2021 03:55:57 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment