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

Paul Kendall paul
Wed Apr 22 08:05:06 CEST 2009


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:
> > Thanks for your comments.
> > Diego, I have made the changes you suggested as well.
>
> Not really...
>
Sorry, thought I had gone through them all.

[...]

>
> > +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.

[...]
>
> Diego

Paul



More information about the ffmpeg-devel mailing list