[coreboot] cbfstool warnings

Myles Watson mylesgw at gmail.com
Fri May 8 21:06:15 CEST 2009


Since cbfstool is so simple right now, it makes sense to have -Werr in
CFLAGS so that we are more careful with the code.  It will be a bigger
pain later.

I didn't want to touch lzma so the tools directory has its own CFLAGS now.

I chose to make all sizes int.  They could have been unsigned int.  If
anyone really cares we can change it.  It just needs to be consistent.

I cleaned up some of the parameters (usage and checking.)  There's no
need to have an empty add call, etc.

Signed-off-by: Myles Watson <mylesgw at gmail.com>

Thanks,
Myles
-------------- next part --------------
A non-text attachment was scrubbed...
Name: misc.diff
Type: text/x-patch
Size: 7868 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090508/4e02588f/attachment.diff>


More information about the coreboot mailing list