[MPlayer-dev-eng] [PATCH] OpenDML AVI2.0 read support

Adam Rice adamrice at ntlworld.com
Fri Feb 6 23:11:52 CET 2004


Quoting D Richard Felker III (dalias at aerifal.cx):
> BTW, when changing them one needs to decide which message level to put
> them at, not just blindly search-and-replace. If you do blind s&r now,
> it would work ok, but then later on when someone wants to fix the
> message levels, it'll be difficult to figure out which messages
> already have intentionally-assigned levels and which don't.

How about creating a #define FIXME_MESSAGE_LEVEL, and then setting them all to
that? Then you can do the search & replace now, and then people can go through
and manually correct the message levels as and when they have the tuits. Then
you could at least add

#define printf dont_use_printf_use_mp_msg

in one of the header files and avoid the problem being re-introduced in
future.

Adam

-- 
Adam Rice -- adamrice at ntlworld.com -- Blackburn, Lancashire, England




More information about the MPlayer-dev-eng mailing list