[coreboot-gerrit] Patch merged into coreboot/master: e44a89f amd/agesa/f15/Lib/amdlib.c: Add missing breaks to switch statement

gerrit at coreboot.org gerrit at coreboot.org
Fri Jun 28 01:35:21 CEST 2013


the following patch was just integrated into master:
commit e44a89f6fd4f421f28ef766b380dca112bf4aea2
Author: Bruce Griffith <Bruce.Griffith at se-eng.com>
Date:   Tue Jun 25 14:14:14 2013 -0600

    amd/agesa/f15/Lib/amdlib.c: Add missing breaks to switch statement
    
    Static analysis often flags case statements that do not include
    a terminating "break;" statement. Eclipse's CODAN is an example
    of this.  This changelist modifies amdlib.c to terminate
    case statements with "break;".
    
    Change-Id: I3d43acaf64e2e2d9717421cb547fec35e582cf8b
    Signed-off-by: Bruce Griffith <Bruce.Griffith at se-eng.com>
    Reviewed-on: http://review.coreboot.org/3539
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See http://review.coreboot.org/3539 for details.

-gerrit



More information about the coreboot-gerrit mailing list