[Mplayer-cvslog] CVS: main/libmpdemux demux_ogg.c,1.49,1.50

The Wanderer inverseparadox at comcast.net
Fri May 7 07:37:19 CEST 2004


Moritz Bunkus CVS wrote:

> --- demux_ogg.c	29 Apr 2004 21:01:09 -0000	1.49
> +++ demux_ogg.c	5 May 2004 21:05:36 -0000	1.50
> @@ -440,7 +440,7 @@
>        if (os->text && d->sub->id == -1 && demux_ogg_check_lang(val, dvdsub_lang))
>        {
>  	d->sub->id = id;
> -	mp_msg(MSGT_DEMUX, MSGL_V, "Stream language matched -slang: %s\n", val);
> +        mp_msg(MSGT_DEMUX, MSGL_V, "OGG demuxer: Displaying the subtitle stream with the id %d which matched -slang %s\n", id, val);

Suggest

"Ogg demuxer: Stream ID %d matched -slang %s\n"

or

"Ogg demuxer: Displaying subtitle stream %d, which matched -slang %s\n"

because the new form looks toe verbose to me, and either of these seems
more grammatical. Several different forms of the latter are competing in
my head to be presented, but that one is as good an arbitrary choice as
any.

This line also has an indentation change, which may or may not have been
intentional.

-- 
       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.




More information about the MPlayer-cvslog mailing list