Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 16 Sep 2001 04:22:36 +1000 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        "David E. O'Brien" <obrien@FreeBSD.org>
Cc:        <cvs-committers@FreeBSD.org>, <cvs-all@FreeBSD.org>
Subject:   Re: cvs commit: src/share/man/man9 style.9
Message-ID:  <20010916041723.A29404-100000@delplex.bde.org>
In-Reply-To: <200109151647.f8FGlOl22240@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 15 Sep 2001, David E. O'Brien wrote:

> obrien      2001/09/15 09:47:24 PDT
>
>   Modified files:
>     share/man/man9       style.9
>   Log:
>   [-- Attachment #1 --]
>   [-- Type: text/plain, Encoding: quoted-printable, Size: 0.1K --]
>
>   Hi! How are you?
>
>   I send you this commit log in order to have your advice
>
>   See you later. Thanks

This commit log has some style bugs.

>   [-- Attachment #2: CVS Commit Log.doc --]
>   [-- Type: application/mixed, Encoding: base64, Size: 315K --]
>
>   Update SCM ID guidelines to reflect the newly added __FBSDID macro.
>
>   Revision  Changes    Path
>   1.65      +5 -8      src/share/man/man9/style.9

Likewise for the contents:
- missing blank line after #include.
- spammed vendor id.  Don't change lines containing vendor ids.  It causes
  patch conflicts for new imports and larger diffs for change reviews.
- line containing $FreeBSD$ longer than before.  The $FreeBSD was indented
  only 2 spaces to gives some chance of the line not being too long.

Bruce


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010916041723.A29404-100000>