qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] Document QEMU coding style (v2)


From: malc
Subject: Re: [Qemu-devel] [PATCH] Document QEMU coding style (v2)
Date: Wed, 1 Apr 2009 19:35:06 +0400 (MSD)

On Wed, 1 Apr 2009, Edgar E. Iglesias wrote:

> On Tue, Mar 31, 2009 at 08:23:06PM +0300, Avi Kivity wrote:
> > With the help of some Limoncino I noted several aspects of the QEMU coding
> > style, particularly where it differs from the Linux coding style as many
> > contributors work on both projects.
> > 
> > Signed-off-by: Avi Kivity <address@hidden>
> > 
> > Changes from v1:
> >   - s/Qemu/QEMU/
> >   - s/Posix/POSIX/
> >   - note that the _t suffix is not POSIX compliant, and will be changed
> >   - discourage eol whitespace
> >   - be strict about the 80 char line width
> > ---
> >  CODING_STYLE |   78 
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 files changed, 78 insertions(+), 0 deletions(-)
> >  create mode 100644 CODING_STYLE
> > 
> > diff --git a/CODING_STYLE b/CODING_STYLE
> > new file mode 100644
> > index 0000000..1ab13b6
> > --- /dev/null
> > +++ b/CODING_STYLE
> > @@ -0,0 +1,78 @@
> > +Qemu Coding Style
> > +=================
> > +
> > +1. Whitespace
> > +
> > +Of course, the most important aspect in any coding style is whitespace.
> > +Crusty old coders who have trouble spotting the glasses on their noses
> > +can tell the difference between a tab and eight spaces from a distance
> > +of approximately fifteen parsecs.  Many a flamewar have been fought and
> > +lost on this issue.
> > +
> > +QEMU indents are four spaces.  Tabs are never used, except in Makefiles
> > +where they have been irreversibly coded into the syntax by some moron.
> > +Spaces of course are superior to tabs because:
> 
> 
> Hi I'd appreciate it if we removed the insults and the arrogance, i.e
> skip the "moron" and the of courses. Other than that most of this stuff

Seconded.

> looks acceptable to me, although I don't necessarily agree with all of
> it :)

[..snip..]

-- 
mailto:address@hidden




reply via email to

[Prev in Thread] Current Thread [Next in Thread]