Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/23022 )
Change subject: make args for -r/-w/-v non-positional and optional ......................................................................
Patch Set 6:
I'm missing a check for spurious arguments. I have no idea how to implement it (straightforward) together with the non-positional file names. Currently, if you have a typo in your command line (e.g. forget a dash before an option), things may just be ignored...
So why make it non-positional?
Is there a way to change the argument amount on -r/-w/-v since without this change, using -i region:file one always needs a filename next to -r/-w/-v which sort of defeats the purpose of the previous patch.