Date: Sat, 15 Mar 2003 17:38:49 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: des@ofug.org Cc: mdodd@FreeBSD.org, src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/net if_iso88025subr.c Message-ID: <20030315.173849.103229416.imp@bsdimp.com> In-Reply-To: <xzpwuj0citu.fsf@flood.ping.uio.no> References: <200303152237.h2FMbCCZ012310@repoman.freebsd.org> <xzpwuj0citu.fsf@flood.ping.uio.no>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <xzpwuj0citu.fsf@flood.ping.uio.no>
des@ofug.org (Dag-Erling Sm=F8rgrav) writes:
: "Matthew N. Dodd" <mdodd@FreeBSD.org> writes:
: > Log:
: > - Style(9) changes.
: > - Remove unneeded assignment.
: > - Increment if_oerrors as per if_fddisubr.c.
: > - Wrap ISO code with conditional.
: =
: This was the 14th consecutive commit to the same file in under 6
: hours. I think that's some kind of record... Next time, please test=
: before committing.
I don't see Matt's changes as a lack of testing. he's doing what
people have always done. I did it many times in the oldcard code when
I wanted to make each change separately on purpose.
Warner
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?20030315.173849.103229416.imp>