On Sat, Jan 24, 2009 at 06:47:16PM +0100, Sascha Sommer wrote:
>
> 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.
I suggest leaving personal preference aside in favor of a generally
consistent coding style across the project.
Diego
_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
