[FFmpeg-devel] [PATCH] LATM Parser & LATM/AAC Decoder

[Diego Biurrun]

On Wed, Apr 22, 2009 at 06:05:06PM +1200, Paul Kendall wrote:
> On Wednesday 22 April 2009 05:41:12 pm Diego Biurrun wrote:
> > On Wed, Apr 22, 2009 at 04:48:46PM +1200, Paul Kendall wrote:
> >
> > > +static int readAudioSyncStream(struct AACDecoder *parser, GetBitContext *b,
> > > +        int size, uint8_t *payload, int *payloadsize)
> >
> > weird indentation, more below
> 
> Which is the accepted indentation for long function declarations? I looked
> around the rest of the code and it seems that 8 spaces on the second line
> is favored.

I have no idea where you got the idea from, but you cannot have looked
into many files.  You need to *sensibly* indent, not mechanically use a
fixed amount of spaces.  Indent the function arguments to continue at
the column where the function arguments begin:

static int readAudioSyncStream(struct AACDecoder *parser, GetBitContext *b,
                               int size, uint8_t *payload, int *payloadsize)

The indent program does this as well.

Diego