Re: [FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 11:26:56PM +0100, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 08:49:45PM +0100, Reimar Döffinger wrote: > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/lpc.c | 2 +- > > libavcodec/lpc.h | 4 > > 2 files changed, 5 insertions(+), 1 deletion(-) > >

Re: [FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Michael Niedermayer
On Sat, Nov 22, 2014 at 08:49:45PM +0100, Reimar Döffinger wrote: > Signed-off-by: Reimar Döffinger > --- > libavcodec/lpc.c | 2 +- > libavcodec/lpc.h | 4 > 2 files changed, 5 insertions(+), 1 deletion(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0

Re: [FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 08:49:45PM +0100, Reimar Döffinger wrote: > Signed-off-by: Reimar Döffinger Was of course meant to be "reduce stack usage" in subject, fixed locally. I thought about adding a alloc function for LLSModel while at it, but the code changes would have needed to be a bit larger

[FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/lpc.c | 2 +- libavcodec/lpc.h | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/lpc.c b/libavcodec/lpc.c index f54f6f8..deb02e7 100644 --- a/libavcodec/lpc.c +++ b/libavcodec/lpc.c @@ -208,7 +208,7 @@ int ff_lpc_calc