Re: [FFmpeg-devel] [PATCH 3/3] Adds a new hls_flag avg_bw

2018-09-29 Thread Amit Kale
Thanks for your feedback. Will soon make these changes and re-submit. -Amit On Fri, Sep 28, 2018 at 7:02 PM Jeyapal, Karthick wrote: > Please find my comments inlined below. > On 9/28/18 11:36 AM, Amit Kale wrote: > > If this flag is set, AVERAGE-BANDWIDTH value will be added to a master > playl

Re: [FFmpeg-devel] [PATCH 3/3] Adds a new hls_flag avg_bw

2018-09-28 Thread Jeyapal, Karthick
Please find my comments inlined below. On 9/28/18 11:36 AM, Amit Kale wrote: > If this flag is set, AVERAGE-BANDWIDTH value will be added to a master > playlist entry. This flag implies peak_segment_bw. Better to add a code like below to set peak segment bw flag explicitly(with comments) in hls_i

[FFmpeg-devel] [PATCH 3/3] Adds a new hls_flag avg_bw

2018-09-27 Thread Amit Kale
If this flag is set, AVERAGE-BANDWIDTH value will be added to a master playlist entry. This flag implies peak_segment_bw. Signed-off-by: Amit Kale --- doc/muxers.texi | 4 libavformat/dashenc.c | 2 +- libavformat/hlsenc.c | 20 libavformat/hlsplay

[FFmpeg-devel] [PATCH 3/3] Adds a new hls_flag avg_bw

2018-02-12 Thread Amit Kale
If this flag is set, AVERAGE-BANDWIDTH value will be added to a master playlist entry. This flag implies peak_segment_bw. ---  doc/muxers.texi   |  4  libavformat/dashenc.c |  2 +-  libavformat/hlsenc.c  | 20  libavformat/hlsplaylist.c |  6 --  libavf