[flashrom] [PATCH] common print infrastructure

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sat Dec 12 03:58:00 CET 2009


Awesome stuff. A must-have for 0.9.2. It touches a lot of code, so I'd
like to merge it as the last patch before 0.9.2 to minimize conflicts.

Minor comments:
- MSG_NORMAL: Not 100% sure if this name is best (_INFO or _NOTICE or
_DEFAULT maybe).
- flashrom_print.c: Maybe rename it to flashrom_output.c and move stuff
like progress printing in there, but I'm open to leaving it as is.
- Conditional compilation: Should the code in flashrom_print.c be #ifdef
STANDALONE or somesuch?
- Makefile: Did you forget to hook up the new file in Makefile?

Regards,
Carl-Daniel

-- 
Developer quote of the month: 
"We are juggling too many chainsaws and flaming arrows and tigers."





More information about the flashrom mailing list