cvs commit: doc/en_US.ISO8859-1/books/arch-handbook/driverbasics chapter.sgml

Ceri Davies setantae at submonkey.net
Thu Sep 25 08:35:53 PDT 2003


On Thu, Sep 25, 2003 at 05:25:25PM +0200, Josef El-Rayes wrote:
> Ken Smith <kensmith at FreeBSD.org> wrote:
> > - "whitespace change" that coverts existing driver example code
> > to be style(9) compliant, matching format of new driver code
> 
> for me this is strange, i always thought that we do not do patches which
> do whitespace changes only - which does make the repository bigger
> without any real benefit.
>                                                                                          
> is there a special reason why we do this here?

We always make whitespace only changes separately, so that translators know
they can ignore the whitespace commits.

Gratuitous whitespace sweeps are contentious, but not nearly as much as many
small whitespace changes.  The difference with Ken's commit is that if a
commit you have made has made a section look ugly, then you should immediately
follow it up with a whitespace commit to reindent it.

Ceri
-- 
User: DO YOU ACCEPT JESUS CHRIST AS YOUR PERSONAL LORD AND SAVIOR?
Iniaes: Sure, I can accept all forms of payment.
                                           -- www.chatterboxchallenge.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-doc/attachments/20030925/f764e1ae/attachment.bin


More information about the cvs-doc mailing list