Messages in this thread | | | Date | Sun, 28 Jun 2009 11:45:48 -0700 | From | Arjan van de Ven <> | Subject | Re: [PATCH 00/62] Remove unnecessary semicolons |
| |
On Sun, 28 Jun 2009 11:32:34 -0700 (PDT) David Miller <davem@davemloft.net> wrote:
> From: Arjan van de Ven <arjan@infradead.org> > Date: Sun, 28 Jun 2009 10:43:32 -0700 > > > On Sun, 28 Jun 2009 10:16:05 -0700 > > Joe Perches <joe@perches.com> wrote: > > > >> On Sun, 2009-06-28 at 10:11 -0700, Arjan van de Ven wrote: > >> > can you improve your descriptions please? > >> > to at least include the why for a change, not just the how and > >> > what... > >> > >> Is any part of "Remove unnecessary semicolons" unclear? > >> > >> What would you prefer? > >> > > > > your sentence addresses the what question, no the why question > > > I think you're going overboard in these requirements. > > Extraneous semi-colons dirty up the code, make it confusing and make > hard to find bugs more likely to be present or (even worse) get added > unintentionally in the future.
... and this sentence would have been a really good changelog.
-- Arjan van de Ven Intel Open Source Technology Centre For development, discussion and tips for power savings, visit http://www.lesswatts.org
| |