Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-22 Thread Rodger Combs
> On Oct 21, 2014, at 19:19, Carl Eugen Hoyos wrote: > > Rodger Combs gmail.com> writes: > >> This fixes https://trac.ffmpeg.org/ticket/3934 but I'm >> not sure if there was a good reason for this to be >> here to begin with. Perhaps a protection against >> infinite recursion (though I belie

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 01:34:16AM -0500, Rodger Combs wrote: > > > On Oct 19, 2014, at 23:34, Rodger Combs wrote: > > > > > >> On Oct 17, 2014, at 17:40, Michael Niedermayer >> > wrote: > >> > >> On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: >

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-21 Thread Carl Eugen Hoyos
Rodger Combs gmail.com> writes: > This fixes https://trac.ffmpeg.org/ticket/3934 but I'm > not sure if there was a good reason for this to be > here to begin with. Perhaps a protection against > infinite recursion (though I believe EBML_MAX_DEPTH > serves that purpose to some degree)? Is all

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-20 Thread Rodger Combs
> On Oct 19, 2014, at 23:34, Rodger Combs wrote: > > >> On Oct 17, 2014, at 17:40, Michael Niedermayer > > wrote: >> >> On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: >>> On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: > On

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-19 Thread Rodger Combs
> On Oct 17, 2014, at 17:40, Michael Niedermayer wrote: > > On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: >> On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: >>> On Oct 17, 2014, at 01:52, Rodger Combs wrote: > On Oct 17, 2014, at 01:16,

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: > On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: > > > > > On Oct 17, 2014, at 01:52, Rodger Combs wrote: > > > > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs > >> > wrote:

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: > > > On Oct 17, 2014, at 01:52, Rodger Combs wrote: > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs >> > wrote: > >> > >> This fixes https://trac.ffmpeg.org/ticket/3934 > >>

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread wm4
On Fri, 17 Oct 2014 01:55:35 -0500 Rodger Combs wrote: > > > On Oct 17, 2014, at 01:52, Rodger Combs wrote: > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs >> > wrote: > >> > >> This fixes https://trac.ffmpeg.org/ticket/3934 > >>

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-16 Thread Rodger Combs
> On Oct 17, 2014, at 01:52, Rodger Combs wrote: > > >> On Oct 17, 2014, at 01:16, Rodger Combs > > wrote: >> >> This fixes https://trac.ffmpeg.org/ticket/3934 >> , but I'm not sure if there was a good >> reason for this to

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-16 Thread Rodger Combs
> On Oct 17, 2014, at 01:16, Rodger Combs wrote: > > This fixes https://trac.ffmpeg.org/ticket/3934 > , but I'm not sure if there was a good > reason for this to be here to begin with. Perhaps a protection against > infinite recursion (though I believe EBM

[FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-16 Thread Rodger Combs
This fixes https://trac.ffmpeg.org/ticket/3934 , but I'm not sure if there was a good reason for this to be here to begin with. Perhaps a protection against infinite recursion (though I believe EBML_MAX_DEPTH serves that purpose to some degree)? _