Attention is currently required from: Edward O'Callaghan, Alexander Goncharov.
Hello build bot (Jenkins), Edward O'Callaghan, Anastasia Klimchuk,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/66531
to look at the new patch set (#6).
Change subject: cli_classic: wrap options validation in function ......................................................................
cli_classic: wrap options validation in function
Introduce a function that will be responsible for checking the parameters.
This is one of the steps on the way to simplify the main function by using helper functions with descriptive names.
TOPIC=split_main_func TEST=the following scenarios failed as expected
./flashrom -p dummy:emulate=W25Q128FV -r "" ./flashrom -p dummy:emulate=W25Q128FV -l "" ./flashrom -p dummy:emulate=W25Q128FV --fmap-file "" ./flashrom -p dummy:emulate=W25Q128FV --flash-contents "" ./flashrom -p dummy:emulate=W25Q128FV --o ""
Change-Id: Ib5685eb5ee3810ef2efd40a3901340e2ceb229ff Signed-off-by: Alexander Goncharov chat@joursoir.net --- M cli_classic.c 1 file changed, 41 insertions(+), 16 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/31/66531/6