Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 10 Sep 2002 20:15:10 -0700
From:      "David O'Brien" <obrien@FreeBSD.org>
To:        Tony Finch <fanf@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/usr.bin/uudecode uudecode.c
Message-ID:  <20020911031509.GA15389@dragon.nuxi.com>
In-Reply-To: <200209102114.g8ALEE6d063516@freefall.freebsd.org>
References:  <200209102114.g8ALEE6d063516@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Sep 10, 2002 at 02:14:14PM -0700, Tony Finch wrote:
> fanf        2002/09/10 14:14:14 PDT
> 
>   Modified files:
>     usr.bin/uudecode     uudecode.c 
>   Log:
>   Fix style bugs in macros.

Geez this is ridiculous, could you *please* batch changes a little bit
better than this??  You've committed 10 times to uudecode.c in less than
3 hours.  (11 times since Sept 3rd).  This is a single file program of
only 361 lines (wc -l uudecode.c  (this includes comments, blank lines,
everything).


These could all be made in a single commit:
    Style: space after return.
    Remove register keyword.
    Fix style bugs in macros.

P.S. `make buildworld' to test them would also be nice. ;-)

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?20020911031509.GA15389>