From: "Richard B. Johnson" <root@chaos.analogic.com>
To: Timothy Miller <miller@techsource.com>
Cc: Jasper Spaans <jasper@vs19.net>,
Zwane Mwaikambo <zwane@arm.linux.org.uk>,
torvalds@osdl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Change all occurrences of 'flavour' to 'flavor'
Date: Fri, 8 Aug 2003 11:34:31 -0400 (EDT) [thread overview]
Message-ID: <Pine.LNX.4.53.0308081127160.502@chaos> (raw)
In-Reply-To: <3F33BF33.8070601@techsource.com>
On Fri, 8 Aug 2003, Timothy Miller wrote:
>
>
> Jasper Spaans wrote:
> > On Thu, Aug 07, 2003 at 09:42:37PM -0400, Zwane Mwaikambo wrote:
> >
> >
> >>>It changes all occurrences of 'flavour' to 'flavor' in the complete tree;
> >>>I've just comiled all affected files (that is, the config resulting from
> >>>make allyesconfig minus already broken stuff) succesfully on i386.
> >>
> >>Arrrgh! You can't be serious!
> >
> >
> > Yes, I am bloody serious; this patch might look purely cosmetic at first
> > sight.. yet, there's a technical reason for at least one part of it. Grep
> > and see the horror:
> >
> > $ egrep -ni 'flavou?r' fs/nfs/inode.c
> > [snip]
> > 1357: rpc_authflavor_t authflavour;
> > [snip]
>
>
> Ah, for a moment, I was worried that someone was talking about text in
> comments.
>
> Yes, when it comes to spelling of words in variable and type names, I
> think it would be a good idea to be consistent.
>
> What is Linus's preferred spelling? Let's use that.
>
It's a little past April Fool's day. What are you guys
smoking? Next, you'll want to change all the 'i's to
'y's and add 'ing'.
Certainly one might want to change:
int TheVariableThatWillBeUsedAsACounter;
to..
int i;
However, changing the spelling of a variable name is
absurd no matter how you look at it.
Cheers,
Dick Johnson
Penguin : Linux version 2.4.20 on an i686 machine (797.90 BogoMips).
Note 96.31% of all statistics are fiction.
next prev parent reply other threads:[~2003-08-08 15:35 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-08-07 18:00 Jasper Spaans
2003-08-07 22:37 ` Alan Cox
2003-08-08 0:04 ` Martin J. Bligh
2003-08-08 16:54 ` Steven Cole
2003-08-08 5:46 ` [PATCH] Change all occurrences of 'flavour' to 'flavor', torvalds@osdl.org Jasper Spaans
[not found] ` <200308080153.22495.jcwren@jcwren.com>
2003-08-08 5:58 ` [ot] Re: [PATCH] Change all occurrences of 'flavour' to 'flavor' Jasper Spaans
2003-08-08 7:13 ` Jens Axboe
2003-08-08 1:42 ` Zwane Mwaikambo
2003-08-08 6:52 ` Jasper Spaans
2003-08-08 6:57 ` Zwane Mwaikambo
2003-08-08 7:15 ` Jens Axboe
2003-08-08 11:04 ` [OT] " Adrian Bunk
2003-08-08 16:58 ` Svein Ove Aas
2003-08-08 21:51 ` Ian Hastie
2003-08-08 7:33 ` Jasper Spaans
2003-08-08 7:40 ` Jens Axboe
2003-08-08 8:00 ` Jasper Spaans
2003-08-08 8:54 ` Jens Axboe
2003-08-08 7:45 ` Zwane Mwaikambo
2003-08-08 7:07 ` Alexander Winston
2003-08-08 7:31 ` Jasper Spaans
2003-08-08 7:11 ` Tomas Szepe
2003-08-08 15:18 ` Timothy Miller
2003-08-08 15:34 ` Richard B. Johnson [this message]
2003-08-08 15:48 ` Trond Myklebust
2003-08-08 16:42 ` Gene Heskett
2003-08-08 17:03 ` Jasper Spaans
2003-08-08 17:34 ` Alan Cox
2003-08-08 18:02 ` Gene Heskett
2003-08-08 15:50 ` Jasper Spaans
2003-08-08 16:00 ` Richard B. Johnson
2003-08-08 17:18 ` Linus Torvalds
2003-08-08 17:33 ` Jasper Spaans
2003-08-09 0:24 ` Trond Myklebust
2003-08-09 0:41 ` Linus Torvalds
2003-08-09 0:45 ` Jasper Spaans
2003-08-09 19:56 ` [PATCH] Fix up fs/nfs/inode.c wrt flavo[u]r Jasper Spaans
2003-08-09 19:51 ` Zwane Mwaikambo
2003-08-09 21:41 ` [PATCH] Fix up fs/nfs/inode.c wrt flavo[u]r, try 2 Jasper Spaans
2003-08-09 1:04 ` [PATCH] Change all occurrences of 'flavour' to 'flavor' Trond Myklebust
2003-08-09 1:17 ` Ian Hastie
2003-08-09 7:23 ` jw schultz
2003-08-07 20:40 Andries.Brouwer
2003-08-08 8:25 Andries.Brouwer
2004-01-13 13:35 Andries.Brouwer
2004-01-13 13:42 Randal, Phil
2004-01-13 14:07 ` Måns Rullgård
2004-01-13 14:26 Nicholas Berry
2004-01-13 22:46 ` Peter Horton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.LNX.4.53.0308081127160.502@chaos \
--to=root@chaos.analogic.com \
--cc=jasper@vs19.net \
--cc=linux-kernel@vger.kernel.org \
--cc=miller@techsource.com \
--cc=torvalds@osdl.org \
--cc=zwane@arm.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®