[coreboot-gerrit] Patch merged into coreboot/master: b532b12 model_fxx/processor_name.c, hudson/lpc.c: add missing break statements

gerrit at coreboot.org gerrit at coreboot.org
Wed Jul 30 10:35:28 CEST 2014


the following patch was just integrated into master:
commit b532b12b411d315b7799fc00a987bf21deb7e8a3
Author: Daniele Forsi <dforsi at gmail.com>
Date:   Sat Jul 26 10:32:34 2014 +0200

    model_fxx/processor_name.c, hudson/lpc.c: add missing break statements
    
    Found by Cppcheck 1.65. Fixes:
    (warning) Variable 'processor_name_string' is reassigned a value before the old one has been used. 'break;' missing?
    (warning) Variable 'rsize' is reassigned a value before the old one has been used. 'break;' missing?
    
    Change-Id: I4a5c947fd5cc5797eb026475ec7036bc5eaf58db
    Signed-off-by: Daniele Forsi <dforsi at gmail.com>
    Reviewed-on: http://review.coreboot.org/6372
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>


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

-gerrit



More information about the coreboot-gerrit mailing list