Re: [Qemu-devel] Re: CODING_STYLE

["Michael S. Tsirkin" <mst@redhat.com>]

On Thu, Oct 01, 2009 at 10:56:15AM +0200, Kevin Wolf wrote:
> Am 01.10.2009 03:25, schrieb Edgar E. Iglesias:
> > On Wed, Sep 30, 2009 at 07:00:21PM +0200, Juan Quintela wrote:
> >> "Michael S. Tsirkin" <mst@redhat.com> wrote:
> >>
> >> ....
> >>
> >>>> Actually, I tried to tweak 'indent' to match QEMU style without much success.
> >>>>
> >>>> We could also select a new style, like Linux kernel one. I'm also not
> >>>> a great fan of that either. But at least 'indent' could be used very
> >>>> easily and I think there would be other benefits, like reuse of Git
> >>>> hooks, patch checking scripts and Emacs configs. With full 'indent'
> >>>> support all sources could be massaged mechanically to make the switch
> >>>> less painful.
> >>>>
> >>>> Then there are K&R and GNU styles, but I like them even less.
> >>>
> >>> So ... Linux kernel style wins? What do others think?
> >>
> >> Where do I have to sign?
> > 
> > 	Yeah.
> > 
> > I really don't care very much. I've changed my CRIS and MicroBlaze code from
> > style to style a couple of times but the only style regardless of style I find
> > offensive is the:
> > 
> > if (x) {
> >   x=1;
> > } else {
> >   x=0;
> > }
> > 
> > TBH, the extra curley braceys are just pure nonsense.
> 
> Except that when adding another line after x=1; the patch doesn't
> contain a change to the if line (with a possibly long condition and the
> possibility of introducing typos that nobody will see because everybody
> assumes it's just the new brace).

I don't think I saw a bug introduced this way in ages.
The absense of braces is easy enough to spot.

> But then, you could say the ugliness of braces is more important and I
> really don't care too much about the braces. However, I don't think we
> should change the coding style radically like switching to the Linux
> kernel style.
> 
> Kevin

I think this is the only way to get a consistent style, for 3 reasons:
- a lot of people hack at both projects, they just keep mixing styles
- linux has automatic checkers that are maintained and improved over time
- linux style is just better ;)

-- 
MST