[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