Re: [FFmpeg-devel] [PATCH v2] avformat/movenc : Don't write sidx for empty urls

2018-12-04 Thread Jeyapal, Karthick
On 12/1/18 12:01 AM, Michael Niedermayer wrote: > On Fri, Nov 30, 2018 at 05:52:35AM +, Jeyapal, Karthick wrote: > > > > On 11/29/18 11:08 PM, Michael Niedermayer wrote: > > > On Wed, Nov 28, 2018 at 09:45:24PM +0530, Karthick J wrote: > > >> When movenc is used by other segmenting muxers

Re: [FFmpeg-devel] [PATCH v2] avformat/movenc : Don't write sidx for empty urls

2018-11-30 Thread Michael Niedermayer
On Fri, Nov 30, 2018 at 05:52:35AM +, Jeyapal, Karthick wrote: > > On 11/29/18 11:08 PM, Michael Niedermayer wrote: > > On Wed, Nov 28, 2018 at 09:45:24PM +0530, Karthick J wrote: > >> When movenc is used by other segmenting muxers such as dashenc, url field > >> is always empty. > >> In

Re: [FFmpeg-devel] [PATCH v2] avformat/movenc : Don't write sidx for empty urls

2018-11-29 Thread Jeyapal, Karthick
On 11/29/18 11:08 PM, Michael Niedermayer wrote: > On Wed, Nov 28, 2018 at 09:45:24PM +0530, Karthick J wrote: >> When movenc is used by other segmenting muxers such as dashenc, url field is >> always empty. >> In such cases it is better to not write sidx, instead of throwing errors. >> --- >>

Re: [FFmpeg-devel] [PATCH v2] avformat/movenc : Don't write sidx for empty urls

2018-11-29 Thread Michael Niedermayer
On Wed, Nov 28, 2018 at 09:45:24PM +0530, Karthick J wrote: > When movenc is used by other segmenting muxers such as dashenc, url field is > always empty. > In such cases it is better to not write sidx, instead of throwing errors. > --- > libavformat/movenc.c | 3 +++ > 1 file changed, 3

[FFmpeg-devel] [PATCH v2] avformat/movenc : Don't write sidx for empty urls

2018-11-28 Thread Karthick J
When movenc is used by other segmenting muxers such as dashenc, url field is always empty. In such cases it is better to not write sidx, instead of throwing errors. --- libavformat/movenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index