[flashrom] [PATCH] Clean up error code definitions

Stefan Tauner stefan.tauner at student.tuwien.ac.at
Thu Jan 12 02:15:04 CET 2012


*push*

i dont care much for the exact names, but i wish for a few things to be
considered:
 - all of them should have a common prefix. this is mostly a problem
   for the value of "OK". often projects use "ERR_" as prefix which
   leads to ERR_OK macros or similar, which isnt too bad. i would
   prefer that over having different prefixes. if we want a very short
   prefix ERR_ is probably best because most return values will be
   some error. but there is also OK and some warnings, so i would favor
   something more neutral like RET_ or so.
 - do we need a "flashrom" prefix too? iirc these are internal
   values so we can ignore external users, right? we may want to define
   some external values too then? could be postponed...
 - all of them should be somewhat readable in context and have
   approximately the same length.

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner




More information about the flashrom mailing list