[coreboot-gerrit] Patch merged into coreboot/master: commonlib: Fix issues with { } and else
gerrit at coreboot.org
gerrit at coreboot.org
Fri Mar 10 22:27:22 CET 2017
the following patch was just integrated into master:
commit 72c60a472b393363722e0c6093f696ee28a7f56a
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date: Fri Mar 10 10:53:36 2017 -0800
commonlib: Fix issues with { } and else
Fix the following errors and warning detected by checkpatch.pl:
ERROR: open brace '{' following struct go on the same line
ERROR: else should follow close brace '}'
WARNING: else is not generally useful after a break or return
False positives are detected for the following checkpatch.pl error.
ERROR: that open brace { should be on the previous line
These false positives are in cbfs.c for two function definitions.
TEST=Build and run Galileo Gen2
Change-Id: Ic679ff3a2e1cfc0ed52073c20165e05bf21d76f3
Signed-off-by: Lee Leahy <Leroy.P.Leahy at intel.com>
Reviewed-on: https://review.coreboot.org/18750
Reviewed-by: Aaron Durbin <adurbin at chromium.org>
Tested-by: build bot (Jenkins)
See https://review.coreboot.org/18750 for details.
-gerrit
More information about the coreboot-gerrit
mailing list