[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