Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-11-03 Thread Michael Niedermayer
On Thu, Nov 03, 2016 at 05:02:50PM +, Derek Buitenhuis wrote: > On 11/3/2016 4:56 PM, Sasi Inguva wrote: > >> I am just converting the *reference* stream duration from that streams' > > timescale to the timescale of the stream I am setting to. > > > > > > Indeed, I misread this (midnight

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-11-03 Thread Derek Buitenhuis
On 11/3/2016 4:56 PM, Sasi Inguva wrote: >> I am just converting the *reference* stream duration from that streams' > timescale to the timescale of the stream I am setting to. > > Indeed, I misread this (midnight emails FTL; I read time_scale 3 times instead of 2). Anyway, the rest seems OK

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-11-03 Thread Sasi Inguva
On Wed, Nov 2, 2016 at 3:50 PM, Derek Buitenhuis wrote: > On 10/26/2016 7:31 PM, Sasi Inguva wrote: > > According to spec ISO_IEC_15444_12 "For any media stream for which no > segment index is present, referred to as non‐indexed stream, the media > stream associated

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-11-02 Thread Derek Buitenhuis
On 10/26/2016 7:31 PM, Sasi Inguva wrote: > According to spec ISO_IEC_15444_12 "For any media stream for which no segment > index is present, referred to as non‐indexed stream, the media stream > associated with the first Segment Index box in the segment serves as a > reference stream in a

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-11-02 Thread Sasi Inguva
Ping. Sorry for the urgency on this, but this is blocking us internally. Thanks On Mon, Oct 31, 2016 at 10:23 AM, Sasi Inguva wrote: > ping. > > Thanks! > > On Wed, Oct 26, 2016 at 11:31 AM, Sasi Inguva wrote: > >> According to spec ISO_IEC_15444_12 "For

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-31 Thread Sasi Inguva
ping. Thanks! On Wed, Oct 26, 2016 at 11:31 AM, Sasi Inguva wrote: > According to spec ISO_IEC_15444_12 "For any media stream for which no > segment index is present, referred to as non‐indexed stream, the media > stream associated with the first Segment Index box in the

[FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-26 Thread Sasi Inguva
According to spec ISO_IEC_15444_12 "For any media stream for which no segment index is present, referred to as non‐indexed stream, the media stream associated with the first Segment Index box in the segment serves as a reference stream in a sense that it also describes the subsegments for any

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-26 Thread Sasi Inguva
sorry. forgot to remove the debug printf. sending the corrected patch. On Wed, Oct 26, 2016 at 7:47 AM, Michael Niedermayer wrote: > On Tue, Oct 25, 2016 at 11:07:58PM -0700, Sasi Inguva wrote: > > According to spec ISO_IEC_15444_12 "For any media stream for which no >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-26 Thread Michael Niedermayer
On Tue, Oct 25, 2016 at 11:07:58PM -0700, Sasi Inguva wrote: > According to spec ISO_IEC_15444_12 "For any media stream for which no segment > index is present, referred to as non‐indexed stream, the media stream > associated with the first Segment Index box in the segment serves as a >

[FFmpeg-devel] [PATCH] lavf/mov.c: Use the first sidx for tracks without sidx.

2016-10-26 Thread Sasi Inguva
According to spec ISO_IEC_15444_12 "For any media stream for which no segment index is present, referred to as non‐indexed stream, the media stream associated with the first Segment Index box in the segment serves as a reference stream in a sense that it also describes the subsegments for any