linux-kernel - Re: [PATCH] include/linux/regmap.h: fix a couple of typos

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140108214906.GB31886@sirena.org.uk>
Date:	Wed, 8 Jan 2014 21:49:06 +0000
From:	Mark Brown <broonie@...nel.org>
To:	Laszlo Papp <lpapp@....org>
Cc:	linux-doc@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] include/linux/regmap.h: fix a couple of typos

On Wed, Jan 08, 2014 at 09:08:44PM +0000, Laszlo Papp wrote:
> On Wed, Jan 8, 2014 at 9:02 PM, Laszlo Papp <lpapp@....org> wrote:

> > That being said, I will not have time, nor the motivation to argue
> > over such a nuance, so feel free to reject the change.

> Of course, this is just on top of the vim spell checker error as I
> wrote in the commit message...

> Oh yes, and one more factual data in here:

> lpapp ~/Projects/linux-staging $ grep -rn "e\.g\." . | wc -l
> 3447

> lpapp ~/Projects/linux-staging $ grep -rn " eg," | wc -l
> 18

That's not the issue - it's dropping the comma.  It's either "e.g." or
"eg", the comma is a separate thing providing a break between clauses.
Strictly it should have the periods since it is an abbreviation but
their use is more vauge in fixed point text since they look ugly, the
thing that made me complain was that you dropped the comma as well as
substituting in the expanded version.

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)