[flashrom] [PATCH 2/3] Factor out DMI string reading into subfunction
c-d.hailfinger.devel.2006 at gmx.net
Wed Feb 24 16:35:56 CET 2010
On 24.02.2010 14:28, Michael Karcher wrote:
> Signed-off-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>
Looks OK, but I'm in a hurry and couldn't check too thorougly. I need to
regenerate the diff with ignored whitespace to see the real changes.
Will do so later.
Right now, only a small coding style comment: You have
in a few places. AFAIK flashrom coding style has the opening brace on
the same line as the if/do/switch statement. The only "standalone"
opening braces are those of functions.
"I do consider assignment statements and pointer variables to be among
computer science's most valuable treasures."
-- Donald E. Knuth
More information about the flashrom