[FFmpeg-devel] [PATCH] lavf/mpegts: add 0x15 (metadata) entry to the ISO_types array

[Stefano Sabatini]

On date Thursday 2016-04-07 23:02:30 +0200, Michael Niedermayer encoded:
> On Thu, Apr 07, 2016 at 08:43:40PM +0200, Stefano Sabatini wrote:
> > On date Saturday 2016-04-02 15:56:55 +0200, Michael Niedermayer encoded:
> > > On Sat, Apr 02, 2016 at 12:46:57PM +0200, Stefano Sabatini wrote:
> > > > This allows to recognize ID3 packets from the stream type.
> > > > ---
> > > >  libavformat/mpegts.c | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > 
> > > breaks tickets/2579/old_klv_data_stream.mpg
> > > 
> > > "breaks" in the sense of detecting the klv stuff as id3
> > 
> > Previous patch was indeed broken. New patch attached.
> > -- 
> > FFmpeg = Fiendish & Faithful Magical Problematic Educated Gymnast
> 
> >  mpegts.c |    1 +
> >  1 file changed, 1 insertion(+)
> > f0cbe8e9df6ab2e3fb3306a2049f2b31d94aa664  0003-lavf-mpegts-add-ID3-entry-to-the-REGD_types-array.patch
> > From a756486700d520d7c1e50128c731a59177cc3545 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefasab at gmail.com>
> > Date: Thu, 7 Apr 2016 20:40:52 +0200
> > Subject: [PATCH] lavf/mpegts: add ID3 entry to the REGD_types array
> > 
> > This allows to recognize ID3 packets from their corresponding descriptor
> > tag.
> > ---
> >  libavformat/mpegts.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
> > index e44da1f..1258913 100644
> > --- a/libavformat/mpegts.c
> > +++ b/libavformat/mpegts.c
> > @@ -739,6 +739,7 @@ static const StreamType REGD_types[] = {
> >      { MKTAG('D', 'T', 'S', '3'), AVMEDIA_TYPE_AUDIO, AV_CODEC_ID_DTS   },
> >      { MKTAG('H', 'E', 'V', 'C'), AVMEDIA_TYPE_VIDEO, AV_CODEC_ID_HEVC  },
> >      { MKTAG('K', 'L', 'V', 'A'), AVMEDIA_TYPE_DATA,  AV_CODEC_ID_SMPTE_KLV },
> > +    { MKTAG('I', 'D', '3', ' '), AVMEDIA_TYPE_DATA,  AV_CODEC_ID_TIMED_ID3 },
> >      { MKTAG('V', 'C', '-', '1'), AVMEDIA_TYPE_VIDEO, AV_CODEC_ID_VC1   },
> >      { MKTAG('O', 'p', 'u', 's'), AVMEDIA_TYPE_AUDIO, AV_CODEC_ID_OPUS  },
> >      { 0 },
> 
> ok

Applied, thanks.
-- 
FFmpeg = Fancy and Fundamental MultiPurpose Extended Glue