[FFmpeg-devel] [PATCH] adpcm: convert VLAs to malloc/free
Sun Jun 27 03:48:09 CEST 2010
On Sun, Jun 27, 2010 at 02:20:19AM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> > On Sun, Jun 27, 2010 at 01:40:38AM +0100, M?ns Rullg?rd wrote:
> >> Michael Niedermayer <michaelni at gmx.at> writes:
> >> > On Wed, Jun 23, 2010 at 08:49:53PM +0100, Mans Rullgard wrote:
> >> >> ---
> >> >> libavcodec/adpcm.c | 136 +++++++++++++++++++++++++++++++---------------------
> >> >> 1 files changed, 82 insertions(+), 54 deletions(-)
> >> >
> >> > mixes functional and cosmetic changes
> >> No, moving the structs is necessary.
> > that may be but i dont think moving the structs needs the rest of the changes
> Well, no.. but splitting this would be just plain silly. I agree that
> indentation changes make the diff harder to read, but not this.
> > that said the removial of VLAs here does lead to ugly code ... iam not really
> > happy about it.
> The code is exactly as ugly as what the compiler has to do otherwise.
certainly true, but one normally doesnt have to read what the compiler
> What do you suggest instead? The code as is uses up to 12MB of stack
i suggest we ask the author of the code, maybe he has a suggestion
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel