[FFmpeg-devel] [PATCH] avformat/mpegts: recognizes and export private streams

[Wolfgang Lorenz]

One last thing:

Am Fri, 12 Jun 2015 22:06:43 +0200
schrieb Wolfgang Lorenz <wl-chmw at gmx.de>:

> Am Fri, 12 Jun 2015 15:53:41 +0200
> schrieb Michael Niedermayer <michaelni at gmx.at>:
> 
> > On Fri, Jun 12, 2015 at 12:38:19PM +0200, Wolfgang Lorenz wrote:
> > > Am Fri, 12 Jun 2015 00:12:37 +0200
> > > schrieb Wolfgang Lorenz <wl-chmw at gmx.de>:
> > > 
> > > > Am Thu, 11 Jun 2015 23:11:37 +0200
> > > > schrieb Michael Niedermayer <michaelni at gmx.at>:
> > > > 
> > > > > On Thu, Jun 11, 2015 at 10:59:23PM +0200, Wolfgang Lorenz wrote:
> > > > > > Hi Micheal,
> > > > > > 
> > > > > > Am Wed, 10 Jun 2015 23:40:10 +0200
> > > > > > schrieb Michael Niedermayer <michaelni at gmx.at>:
> > > > > > 
> > > > > > > Based on patch by Wolfgang Lorenz <wl-chmw at gmx.de>
> > > > > > > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > > > > > > ---
> > > > > > >  libavformat/mpegts.c |    7 +++++++
> > > > > > >  1 file changed, 7 insertions(+)
> > > > > > > 
> > > > > > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
> > > > > > > index eff6819..7b35d7f 100644
> > > > > > > --- a/libavformat/mpegts.c
> > > > > > > +++ b/libavformat/mpegts.c
> > > > > > > @@ -835,6 +835,13 @@ static int mpegts_set_stream_info(AVStream *st, PESContext *pes,
> > > > > > >          st->codec->codec_id  = old_codec_id;
> > > > > > >          st->codec->codec_type = old_codec_type;
> > > > > > >      }
> > > > > > > +    if ((st->codec->codec_id == AV_CODEC_ID_NONE || st->request_probe == 1) &&
> > > > > > > +        !avcodec_is_open(st->codec) &&
> > > > > > > +        stream_type ==  6) {
I've just seen, mpegts.h contains
  #define STREAM_TYPE_PRIVATE_DATA    0x06

I think, comparing stream_type to STREAM_TYPE_PRIVATE_DATA, makes this
code a little bit more verbose.

> > > > > > > +        st->codec->codec_type = AVMEDIA_TYPE_DATA;
> > > > > > > +        st->codec->codec_id   = AV_CODEC_ID_BIN_DATA;
> > > > > > > +        st->request_probe = 1;
> > > > > > > +    }
> > > > > > >  
> > > > > > >      return 0;
> > > > > > >  }
> > > > > > 
> [...]

That's it. No more questions.

Cheers,
  Wolfgang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150622/b5ff9e57/attachment.asc>