diff mbox

CODING_STYLE: Fix a typo ("Many a flamewar")

Message ID 20160929204647.23526-1-j.neuschaefer@gmx.net (mailing list archive)
State New, archived
Headers show

Commit Message

J. Neuschäfer Sept. 29, 2016, 8:46 p.m. UTC
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 CODING_STYLE | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Maydell Sept. 29, 2016, 10:14 p.m. UTC | #1
On 29 September 2016 at 13:46, Jonathan Neuschäfer
<j.neuschaefer@gmx.net> wrote:
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>  CODING_STYLE | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/CODING_STYLE b/CODING_STYLE
> index e7fde15..a03a994 100644
> --- a/CODING_STYLE
> +++ b/CODING_STYLE
> @@ -9,7 +9,7 @@ patches before submitting.
>  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
> +of approximately fifteen parsecs.  Many flamewars have been fought and
>  lost on this issue.

Should be "Many a flamewar has been fought and lost".
(This is slightly poetic phrasing but it has a nice ring to it;
the complement does have to be singular, not plural, though.)

thanks
-- PMM
J. Neuschäfer Sept. 29, 2016, 11:38 p.m. UTC | #2
On Thu, Sep 29, 2016 at 03:14:45PM -0700, Peter Maydell wrote:
> On 29 September 2016 at 13:46, Jonathan Neuschäfer
> <j.neuschaefer@gmx.net> wrote:
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> > ---
> >  CODING_STYLE | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/CODING_STYLE b/CODING_STYLE
> > index e7fde15..a03a994 100644
> > --- a/CODING_STYLE
> > +++ b/CODING_STYLE
> > @@ -9,7 +9,7 @@ patches before submitting.
> >  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
> > +of approximately fifteen parsecs.  Many flamewars have been fought and
> >  lost on this issue.
> 
> Should be "Many a flamewar has been fought and lost".
> (This is slightly poetic phrasing but it has a nice ring to it;
> the complement does have to be singular, not plural, though.)

Thanks for pointing that out; I didn't know the phrase. I'll update my
patch.


Jonathan Neuschäfer
diff mbox

Patch

diff --git a/CODING_STYLE b/CODING_STYLE
index e7fde15..a03a994 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -9,7 +9,7 @@  patches before submitting.
 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
+of approximately fifteen parsecs.  Many flamewars have been fought and
 lost on this issue.
 
 QEMU indents are four spaces.  Tabs are never used, except in Makefiles