This separates the code for each command in cbfstool. For the good and for the
authorStefan Reinauer <stepan@coresystems.de>
Tue, 22 Sep 2009 15:58:19 +0000 (15:58 +0000)
committerStefan Reinauer <stepan@openbios.org>
Tue, 22 Sep 2009 15:58:19 +0000 (15:58 +0000)
commit3fec29cc963c95d44ff45151a1bc64d8988073ab
tree7dfdc9f802911f736efa6f2f743d1f62235ea59b
parent853270af390f9e200f4a165ec55d3b2fa5246c0b
This separates the code for each command in cbfstool. For the good and for the
bad: It brings a certain amount of code duplication (some of which can be
cleaned up again, or get rid of by proper refactoring).
On the other hand now there's a very simple code flow for each command, rather
than for each operation. ie.

adding a file to a cbfs means:
 - open the cbfs
 - add the file
 - close the cbfs

rather than

open the cbfs:
 - do this for add, remove, but not for create

create a new lar
 - if we don't have an open one yet

add a file:
 - if we didn't bail out before

close the file:
 - if we didn't bail out before

The short term benefit is that this fixes a problem where cbfstool was trying
to add a file if you gave a non-existing command because it bailed out on
known, not on unknown commands.

Signed-off-by: Stefan Reinauer <stepan@coresystems.de>
Acked-by: Peter Stuge <peter@stuge.se>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4654 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
util/cbfstool/cbfstool.c