[FFmpeg-devel] [PATCH] lavc/lavf: transmit stream_id information for mpegts KLV data packets

Stefano Sabatini stefasab at gmail.com
Tue Feb 23 18:59:07 CET 2016


On date Saturday 2016-02-20 20:25:18 +0100, Stefano Sabatini encoded:
> On date Saturday 2016-02-13 21:06:23 +0100, Marton Balint encoded:
> > 
> > On Sat, 13 Feb 2016, Stefano Sabatini wrote:
> > 
> > >>Subject: [PATCH] lavc/lavf: transmit stream_id information for mpegts KLV data
> > >> packets
> > >>
> > 
> > [...]
> > 
> > >Someone else wants to have a look, or should I push as is?
> > 
> > I had a look at the mpegtsenc part as well:
> > 
> > >--- a/libavformat/mpegtsenc.c
> > >+++ b/libavformat/mpegtsenc.c
> > 
> > [...]
> > 
> > >@@ -1451,6 +1456,15 @@ static int mpegts_write_packet_internal(AVFormatContext *s, AVPacket *pkt)
> > >     const int64_t delay = av_rescale(s->max_delay, 90000, AV_TIME_BASE) * 2;
> > >     int64_t dts = pkt->dts, pts = pkt->pts;
> > >     int opus_samples = 0;
> > >+    int side_data_size;
> > >+    char *side_data = NULL;
> > >+    int stream_id = 0;
> > 
> 
> > Why don't you use -1 as undefined? Theoretically stream_id can be
> > zero as well, can't it?
> 
> Good idea. Changed locally.
> 
> > Otherwise it is LGTM. I think you waited long enough, feel free to push.
> 
> Will push in a few days, thanks.

Finally applied.
-- 
FFmpeg = Fabulous and Fantastic Moronic Powered Elastic Guide


More information about the ffmpeg-devel mailing list