[coreboot-gerrit] Patch merged into coreboot/master: cbgfx: make the code more descriptive

gerrit at coreboot.org gerrit at coreboot.org
Tue Oct 27 15:22:36 CET 2015


the following patch was just integrated into master:
commit 9aed1465d70c32e186c2c65e83240630453bf017
Author: Daisuke Nojiri <dnojiri at chromium.org>
Date:   Thu Oct 8 14:03:56 2015 -0700

    cbgfx: make the code more descriptive
    
    This change makes the code in graphics.c more descriptive and readable.
    Especially, it makes expressions for scale calculation look what they
    are meant to do. It also includes:
    
    - Rename variables (struct fraction, dim_org, etc.) for more consistency
    - Add more input validation (div-by-zero, etc.)
    
    BUG=chromium:502066
    BRANCH=master
    TEST=Tested on Samus
    CQ-DEPEND=CL:304860
    
    Change-Id: I2694912bb7b6017d5655de2fd655b95432addb22
    Signed-off-by: Patrick Georgi <pgeorgi at google.com>
    Original-Commit-Id: 0863dc3ee925d3a05c83c66397b19a57f5478ef3
    Original-Change-Id: Id8e349b8e09082fb84c3e1a984617f916e16c518
    Original-Signed-off-by: Daisuke Nojiri <dnojiri at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/304861
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/11928
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list