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

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 935 bytes --]

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@kde.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.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]