[FFmpeg-devel] [PATCH] IFF: Add the HAM stuff

[Michael Niedermayer]

On Sun, May 16, 2010 at 01:03:34AM +0200, Stefano Sabatini wrote:
> On date Sunday 2010-05-16 00:40:21 +0200, Sebastian Vater encoded:
> > Stefano Sabatini a ?crit :
> > >> +/**
> > >> + * Extracts the IFF extra context and updates internal
> > >> + * decoder structures.
> > >> + *
> > >> + * @param avctx the AVCodecContext where to extract extra context to
> > >> + * @param avpkt the AVPacket to extract extra context from
> > >> + *
> > >>     
> > >
> > > Nit+++: no need for this empty newline
> > >   
> > 
> > Which empty new line? They're just cosmetics like in the other
> > functions, too. So you see return value separated to args.
> 
> Well it's a level 3 nit, so it's really not important if you prefer

i thought nit+++ is a syntax error ;)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100516/f429fdd8/attachment.pgp>