Re: [FFmpeg-devel] [PATCH] vorbisenc: avoid large stack allocation.

2014-09-03 Thread Reimar Döffinger
On Wed, Sep 03, 2014 at 02:48:41AM +0200, Michael Niedermayer wrote: > On Wed, Sep 03, 2014 at 12:25:17AM +0200, Reimar Döffinger wrote: > > Code is only used during initialization, so malloc/free > > should be fine to use. > > > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/vorbisenc

Re: [FFmpeg-devel] [PATCH] vorbisenc: avoid large stack allocation.

2014-09-02 Thread Michael Niedermayer
On Wed, Sep 03, 2014 at 12:25:17AM +0200, Reimar Döffinger wrote: > Code is only used during initialization, so malloc/free > should be fine to use. > > Signed-off-by: Reimar Döffinger > --- > libavcodec/vorbisenc.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) LGTM [...] --

[FFmpeg-devel] [PATCH] vorbisenc: avoid large stack allocation.

2014-09-02 Thread Reimar Döffinger
Code is only used during initialization, so malloc/free should be fine to use. Signed-off-by: Reimar Döffinger --- libavcodec/vorbisenc.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c index 0fb7190..0f78d95 100644 --- a