[FFmpeg-devel] [PATCH 5/7] ffmpeg: dynamically allocate stream_maps

[Måns Rullgård]

Aurelien Jacobs <aurel at gnuage.org> writes:

> On Wed, Aug 18, 2010 at 10:57:41PM +0200, Michael Niedermayer wrote:
>> On Fri, Aug 13, 2010 at 08:24:18PM +0200, Aur?lien Jacobs wrote:
>> > From: Aurelien Jacobs <aurel at gnuage.org>
>> > 
>> > ---
>> >  ffmpeg.c |    8 +++++---
>> >  1 files changed, 5 insertions(+), 3 deletions(-)
>> > 
>> > diff --git a/ffmpeg.c b/ffmpeg.c
>> > index dcaaecf..9e55f01 100644
>> > --- a/ffmpeg.c
>> > +++ b/ffmpeg.c
>> > @@ -117,8 +117,8 @@ static AVCodec **output_codecs = NULL;
>> >  static int nb_output_files = 0;
>> >  static int nb_output_codecs = 0;
>> >  
>> > -static AVStreamMap stream_maps[MAX_FILES*MAX_STREAMS];
>> > -static int nb_stream_maps;
>> > +static AVStreamMap *stream_maps = NULL;
>> 
>> > +static int nb_stream_maps = 0;
>> 
>> unneeded
>
> It is actually used. Hidden behind the GROW_ARRAY() macro :-(
> So yes, I understand why you don't like such kind of macro...

He meant the =0 initialiser is unnecessary.

-- 
M?ns Rullg?rd
mans at mansr.com