Attention is currently required from: Edward O'Callaghan, Alexander Goncharov.

Alexander Goncharov uploaded patch set #6 to this change.

View Change

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

To view, visit change 66531. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib5685eb5ee3810ef2efd40a3901340e2ceb229ff
Gerrit-Change-Number: 66531
Gerrit-PatchSet: 6
Gerrit-Owner: Alexander Goncharov <chat@joursoir.net>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Attention: Alexander Goncharov <chat@joursoir.net>
Gerrit-MessageType: newpatchset