[MPlayer-dev-eng] last printf in libaf
Reynaldo H. Verdejo Pinochet
reynaldo at opendot.cl
Wed Mar 15 01:58:21 CET 2006
On Mon, Mar 13, 2006 at 11:15:23PM -0800, Corey Hickey wrote:
> Reynaldo H. Verdejo Pinochet wrote:
> > Hope is ok to commit, is the last one on libaf
>
> [cut]
>
> > +// af_format.c
> > +
> > +#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.
Hi Corey, I didnt get to read this message before commiting :/ there
where problems with the ml. I agree, at least the message needs to be
rethinkered/rewrited, could you please take care of the modification?
Regards
Reynaldo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060314/73e4e9c9/attachment.pgp>
More information about the MPlayer-dev-eng
mailing list