| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 6 Mar 2008 23:03:10 +0030 From: "Paolo Ciarrocchi" <paolo.ciarrocchi@...il.com> To: "Andy Whitcroft" <apw@...dowen.org> Cc: linux-kernel@...r.kernel.org Subject: Re: [PATCH] checkpatch: Make some text more consinstent and informative On 3/6/08, Andy Whitcroft <apw@...dowen.org> wrote: > On Wed, Mar 05, 2008 at 09:49:12PM +0100, Paolo Ciarrocchi wrote: > > Hi Andy, > > > > Now messages about missing spaces or not needed spaces are in the format: > > space is required > > space is not required > > > > Signed-off-by: Paolo Ciarrocchi <paolo.ciarrocchi@...il.com> > > The current version (which your patch has crossed with) has these changed > to "required" and "prohibited" respectivly, to remove the ambiguity > introduced by > tehe "no spaces X" form. > > I did consider and reject the "is not required" form. In english this > does not correct carry the "must not be present" meaning, it more means > "may or may not be present at your option". Which is not what we are > trying to say either. yes, i tend to agree. > Hopefully the new wording is clearer to native and non-native english > speakers alike. i had a look at the next branch, i really like the new output. Very nice work. > Thanks for doing the patch even so. you are welcome, i'll keep tracking the next branch. ciao, -- Paolo http://paolo.ciarrocchi.googlepages.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists