[MPlayer-dev-eng] last printf in libaf

Diego Biurrun diego at biurrun.de
Tue Mar 14 23:50:59 CET 2006


On Mon, Mar 13, 2006 at 11:15:23PM -0800, Corey Hickey wrote:
> Reynaldo H. Verdejo Pinochet wrote:
> > +
> > +#define MSGTR_AF_FORMAT_UnknownSpecial "Unknown special\n"
> 
> My first thought was, "what does that mean?" I took a look at the code,
> and thought saying "unknown format" or maybe "unknown special format"
> would be better.
> 
> Then, I saw that it's in af_fmt2str(), which looks like it converts a
> numeric format into a string for human consumption. Wouldn't it be
> better to use snprintf() to put that message into the string so it gets
> printed in the proper context?
> 
> I would prefer the attached patch.
> 
> 
> Right now we would get something like:
> ...
> Unknown special
> alsa-init: format little-endian not supported by hardware, trying default
> ...
> 
> Using snprintf() would give us something like this:
> ...
> alsa-init: format little-endian unknown-format not supported by
> hardware, trying default
> ...

Please commit, this is a huge improvement..

Diego




More information about the MPlayer-dev-eng mailing list