[MPlayer-dev-eng] [PATCH] libmpdemux/demux_audio: Skip ID3v2.4 tags.

[Benoît Thébaudeau]

Reimar,

> > diff -Nrdup mplayer.orig/libmpdemux/demux_audio.c
> > mplayer/libmpdemux/demux_audio.c
> > --- mplayer.orig/libmpdemux/demux_audio.c	2012-01-27
> > 13:42:31.522983827 +0100
> > +++ mplayer/libmpdemux/demux_audio.c	2012-01-27 13:55:08.946980169
> > +0100
> > @@ -441,12 +441,12 @@ static int demux_audio_open(demuxer_t* d
> >      free(mp3_found);
> >      mp3_found = NULL;
> >      if(s->end_pos && (s->flags & MP_STREAM_SEEK) ==
> >      MP_STREAM_SEEK) {
> > -      stream_seek(s,s->end_pos-128);
> > +      stream_seek(s,demuxer->movi_end-128);
> 
> Semantics-wise, it doesn't really make sense to check
> for s->end_pos but then use demuxer->movi_end.
> Also I realize that it would probably make sense to check
> for it >/>= 128.

Done in the attached patch.

I have also attached the same patch merged with the previous one if you prefer
a single patch for these changes.

Benoît
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1-demux-audio-use-movi-end.patch
Type: text/x-patch
Size: 4228 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120203/964200fd/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1-demux-audio-skip-id3v2.4-merged.patch
Type: text/x-patch
Size: 5982 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120203/964200fd/attachment-0003.bin>