[coreboot-gerrit] Patch merged into coreboot/master: siemens/mc_tcu3: Remove unneeded variable assignment

gerrit at coreboot.org gerrit at coreboot.org
Mon May 9 05:28:18 CEST 2016


the following patch was just integrated into master:
commit 78021190058462bf2ed7da33d3327bd9ebc5ab82
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat May 7 08:18:18 2016 +0200

    siemens/mc_tcu3: Remove unneeded variable assignment
    
    Assigning the value `1` to `status` in the default branch of the switch
    statement is not needed, as the stored value is overwritten before it
    can be used.
    
    Change-Id: I532b0e217ff4ed315cd30b08d339c755c6df7539
    Found-by: Coverity, CID 1355008: Code maintainability issues (UNUSED_VALUE))
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-on: https://review.coreboot.org/14699
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See https://review.coreboot.org/14699 for details.

-gerrit



More information about the coreboot-gerrit mailing list