[FFmpeg-devel] [PATCH] lavc/lavf: transmit stream_id information for mpegts KLV data packets
    Marton Balint 
    cus at passwd.hu
       
    Sat Feb 13 21:06:23 CET 2016
    
    
  
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?
Otherwise it is LGTM. I think you waited long enough, feel free to push.
Regards,
Marton
    
    
More information about the ffmpeg-devel
mailing list