[FFmpeg-soc] [soc]: r3964 - in wmapro: wma3.h wma3data.h wma3dec.c

Sascha Sommer saschasommer at freenet.de
Sat Jan 24 18:47:16 CET 2009


Hi,

On Samstag, 17. Januar 2009, Diego Biurrun wrote:
> On Sat, Jan 17, 2009 at 12:56:08PM -0800, Baptiste Coudurier wrote:
> > faust3 wrote:
> > > [...]
> > >
> > >          if ( idx == 126 )
> > >          {
> > >            while(i < 4){
> >
> > You may need to uniformize '{' placement, also get rid of these spaces
> > before and after parentheses.
>
> I would suggest K&R style, which has spaces after if/while:
>
>     if (idx == 126) {
>         while (i < 4) {
>

I prefere them without spaces.
Anyway I'll try to fix this before I submit the first patch.

Regards

Sascha





More information about the FFmpeg-soc mailing list