[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