[flashrom] [PATCH] Kill globals, initialize programmer-related variables explicitly

Michael Karcher flashrom at mkarcher.dialup.fu-berlin.de
Sat Jul 3 11:41:52 CEST 2010


Am Freitag, den 02.07.2010, 21:36 +0200 schrieb Carl-Daniel Hailfinger:
> > OK, so we keep this as-is now. I still think that we can change even now
> > how the variable is initialized. Current state is that the variable is
> > initalized from the command line parser (which is client code). Even if
> > it stays a global variable, I think we should have the client pass the
> > pointer through programmer_init into the library than having it directly
> > access global variables, but again, we don't have to beat this horse to
> > death. I just want to make the point clear.
> >   
> programmer_param is now a parameter of programmer_init. The variable is
> still global, though, but at least a frontend doesn't have to set it
> directly anymore.

Great. Each global variable removed fromthe frontend->backend interface
is a win for libflashrom.

Patch looks fine now on skimming over it, I did intense review on the
previous iterations.

Acked-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>

Regards,
  Michael Karcher





More information about the flashrom mailing list