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

[Michael Niedermayer]

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) {
> > +        st->codec->codec_type = AVMEDIA_TYPE_DATA;
> > +        st->codec->codec_id   = AV_CODEC_ID_BIN_DATA;
> > +        st->request_probe = 1;
> > +    }
> >  
> >      return 0;
> >  }
> 
> First things first: It's breaking fate-acodec-s302m for me.

make sure you use latest git master of ffmpeg, there was a commit
rather recently that is needed

if that doesnt work with your file then please provide that file


[...]
> If in your patch the request_probe value is set to a higher value, like
> e.g. 10, avformat_find_stream_info() behaves nicely and I do not need
> to reset the value manually. fate-acodec-s302m is still broken, though.

yes, the value will need finetuning, whats the more or less lowest
that results in 100% reliable detection ?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150611/a531bfc8/attachment.asc>