[PATCH] Re: [MPlayer-dev-eng] MSGTR messages for vid/aid in non-verbose output?
Andrew Savchenko
Bircoph at list.ru
Mon Dec 11 20:16:03 CET 2006
On 4 Dec 2006 11:54, Diego Biurrun wrote:
> On Sun, Dec 03, 2006 at 05:56:25PM +0300, Andrew Savchenko wrote:
> > On 2 Dec 2006 23:44, Diego Biurrun wrote:
> > > On Tue, Nov 21, 2006 at 03:03:53PM +0300, Andrew Savchenko
wrote:
> > So my suggestion is to add appropriate messages directly into
> > demuxer.c, so they will be uniform despite of the demuxer
> > type. IMHO this will be usefull to user. OTOH this may make an
> > overhead of usefull information in the cases when format's
> > dexuxer provides this information also (such as demux_ogg and
> > demux_mkv). Therefore must be some check for demuxer type in
> > order to realize necessity of such message. I made this check
> > to skip printing message in the case of ogg or mkv demuxer.
>
> The patch seems OK for a first draft, but the special-casing for
> Ogg and Matroska is ugly. Possibly those two demuxers need to
> be adapted.
I've adapt them, sorry for delay. Also I adapted dvd's demuxing in
stream/stream_dvd.c as Sebasian earlier suggested. But such kind
of changes takes a price: there are two informational lines about
track for these formats, not one; e.g.:
---
[demux] Subtitle stream detected, -sid 0
[mkv] Track ID 4: subtitles (S_TEXT/SSA), -slang eng
---
instead of
---
[mkv] Track ID 4: subtitles (S_TEXT/SSA), -sid 0, -slang eng
---
> > Well, patch attached. The second one is russian translation
> > for new messages.
>
> You can leave out the Russian translation for now, this will be
> taken care of later.
ok
I separate changes into two paches:
1st) Add general info in demuxer.c and change behavior for ogg and
mkv demuxers + update help_mp-en.h for new demuxer messages.
Patch was tested for both *.ogm and *.mkv as well as for ordinary
avis (all files with multiple streams, of course).
2nd) Change behavior of dvd stream demuxer, this requieres changes
in help_mp* files since number of mp_msg arguments was changed.
Thus I separate this patch. Also it figured out that I have no
plain dvds (all was converted to mpeg4/x264) so I can't test dvd
stream demuxer output behavior. Should work fine, but please, test
with some dvds.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20061211/b888d8fd/attachment.pgp>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: demuxer.patch
Type: text/x-diff
Size: 9236 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20061211/b888d8fd/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: stream_dvd.patch
Type: text/x-diff
Size: 13632 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20061211/b888d8fd/attachment-0001.patch>
More information about the MPlayer-dev-eng
mailing list