On Sat, 10 Aug 2013 17:43:15 +0200 Carl-Daniel Hailfinger <[email protected]> wrote:
> An unused programmer parameter is a sign that the user wanted to either > do something not supported by the programmer or misspelled a parameter > which may be essential for the given programmer. Aborting is the only > safe choice. > > Signed-off-by: Carl-Daniel Hailfinger <[email protected]> > > Index: flashrom-abort_unused_programmer_param/flashrom.c > =================================================================== > --- flashrom-abort_unused_programmer_param/flashrom.c (Revision 1706) > +++ flashrom-abort_unused_programmer_param/flashrom.c (Arbeitskopie) > @@ -389,13 +389,14 @@ > programmer_may_write = 1; > > programmer_param = param; > - msg_pdbg("Initializing %s programmer\n", > - programmer_table[programmer].name); > + msg_pdbg("Initializing %s programmer\n", > programmer_table[programmer].name); > ret = programmer_table[programmer].init(); > if (programmer_param && strlen(programmer_param)) { > - msg_perr("Unhandled programmer parameters: %s\n", > - programmer_param); > - /* Do not error out here, the init itself was successful. */ > + msg_perr("Unhandled programmer parameters: %s\n", > programmer_param); > + msg_perr("Aborting.\n"); Considering pro and contra arguments for (not) combining these two lines, I am fine with it.</blah :)> > + /* Do not overwrite any error code from programmer init. */ > + if (!ret) > + ret = ERROR_FATAL; > } > return ret; > } > IMHO the if is unnecessary, not beneficial and should be gone. In any case it is Acked-by: Stefan Tauner <[email protected]> and I want that in 0.9.7. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner _______________________________________________ flashrom mailing list [email protected] http://www.flashrom.org/mailman/listinfo/flashrom
