[MPlayer-dev-eng] [PATCH] printf -> mp_msg conversion (etc.), second patches
The Wanderer
inverseparadox at comcast.net
Tue Aug 24 22:37:01 CEST 2004
The Wanderer wrote:
> Attila Kinali wrote:
>
>> On Wed, Aug 18, 2004 at 04:57:58PM -0400, The Wanderer wrote:
>>
>>> A little more work on the conversion. These patches cover
>>> mplayer.c, and assume that the syntax-change patch from the
>>> previous set has not been applied but that the others have.
>
>>> - if(how) mp_msg(MSGT_CPLAYER,MSGL_INFO,MSGTR_Exiting,mp_gettext(how));
>>> - mp_msg(MSGT_CPLAYER,MSGL_DBG2,"max framesize was %d bytes\n",max_framesize);
>>> + if(how) mp_msg(MSGT_CPLAYER,MSGL_INFO,MSGTR_ExitingHow,mp_gettext(how));
>>> + mp_msg(MSGT_CPLAYER,MSGL_DBG2,MaxFramesizeWas,max_framesize);
>>
>> Please dont add DBG2+ to the help-*.h, they are of no use for the
>> casual user but create unnecessary work for the translators as
>> these messages should always be printed in english.
>
> Acknowledged; will change, and resend (probably) shortly.
Revised patches attached; I've got the nagging feeling I've messed
something up, but I can't find anything obvious.
The recent commit, however, inadvertently broke the compile; one #define
was not committed to the help file, but its corresponding string was
replaced in mencoder.c. The help-file patch in this pair also includes
that line; if in fact the string should not be moved, that's fine, but
in that case it needs to be put back in its original place (and I should
be informed for future reference).
--
The Wanderer
Warning: Simply because I argue an issue does not mean I agree with any
side of it.
A government exists to serve its citizens, not to control them.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: mplayer_messages.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040824/96b40191/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: MSGTR_updates.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040824/96b40191/attachment.asc>
More information about the MPlayer-dev-eng
mailing list