Re: [Qemu-devel] Re: CODING_STYLE

[Michael S. Tsirkin]

On Thu, Oct 01, 2009 at 11:46:16AM +0200, Gleb Natapov wrote:
> On Thu, Oct 01, 2009 at 11:02:59AM +0200, Michael S. Tsirkin wrote:
> > 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" <address@hidden> 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.
> > 
> With modern editors that indent automatically it's impossible not 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 ;)
> > 
> Real men's editor (TM) is able to switch style depending on what you
> edit.
> --
>                       Gleb.