[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, u
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix) |
Date: |
Wed, 30 Sep 2009 15:51:42 +0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Tue, Sep 29, 2009 at 09:15:04PM +0300, Blue Swirl wrote:
> On Tue, Sep 29, 2009 at 7:11 PM, Michael S. Tsirkin <address@hidden> wrote:
> > On Thu, Sep 24, 2009 at 11:13:38PM +0300, Blue Swirl wrote:
> >> >> As this is not a clean revert anyway, please don't revert the part
> >> >> changing
> >> >> if (x)
> >> >> y;
> >> >> else
> >> >> z;
> >> >>
> >> >> to
> >> >>
> >> >> if (x) {
> >> >> y;
> >> >> } else {
> >> >> z;
> >> >> }
> >> >
> >> > Did I mention I hate this style? But okay :).
> >>
> >> I'm not a great fan of this either, but this is the CODING_STYLE.
> >
> > Maybe we can all vote to change this?
>
> 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?
--
MST
- [Qemu-devel] [PATCHv2] qemu: target library, use it in msix, Michael S. Tsirkin, 2009/09/23
- [Qemu-devel] Re: [PATCHv2] qemu: target library, use it in msix, Blue Swirl, 2009/09/24
- [Qemu-devel] Re: [PATCHv2] qemu: target library, use it in msix, Michael S. Tsirkin, 2009/09/24
- [Qemu-devel] Re: [PATCHv2] qemu: target library, use it in msix, Blue Swirl, 2009/09/24
- [Qemu-devel] CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Michael S. Tsirkin, 2009/09/29
- [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Blue Swirl, 2009/09/29
- [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix),
Michael S. Tsirkin <=
- Re: [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), malc, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Avi Kivity, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Blue Swirl, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Anthony Liguori, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Michael S. Tsirkin, 2009/09/30
- [Qemu-devel] Re: CODING_STYLE, Juan Quintela, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE, Edgar E. Iglesias, 2009/09/30
- Re: [Qemu-devel] Re: CODING_STYLE, Markus Armbruster, 2009/09/30
- [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Paolo Bonzini, 2009/09/30
- [Qemu-devel] Re: CODING_STYLE (was Re: [PATCHv2] qemu: target library, use it in msix), Michael S. Tsirkin, 2009/09/30