Date: Sun, 19 Dec 1999 13:56:46 -0800 (PST) From: "Rodney W. Grimes" <freebsd@gndrsh.dnsmgr.net> To: green@FreeBSD.org (Brian Fundakowski Feldman) Cc: wollman@khavrinen.lcs.mit.edu (Garrett Wollman), cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/net if_ethersubr.c if_fddisubr.c if_iso88025subr.c src/sys/netinet raw_ip.c udp_usrreq.c Message-ID: <199912192156.NAA38840@gndrsh.dnsmgr.net> In-Reply-To: <Pine.BSF.4.10.9912190046360.945-100000@green.dyndns.org> from Brian Fundakowski Feldman at "Dec 19, 1999 00:47:52 am"
next in thread | previous in thread | raw e-mail | index | archive | help
> On Sat, 18 Dec 1999, Garrett Wollman wrote: > > > <<On Sat, 18 Dec 1999 17:55:38 -0800 (PST), Brian Feldman <green@FreeBSD.org> said: > > > > > Log: > > > M_PREPEND-related cleanups (unregisterifying struct mbuf *s). > > > > Reviewed by: ???? > > I'm pretty certain that I don't have a sign on me that says "kick me", or > "I cannot commit a simple, thoroughly tested change without a review." We all have worn that sign at one time or another, you know those collections of little orange cones in the corner we all have. Well, to reduce the rate of growth of those it is usually best to drop the attitude ``I am above error in this simple case'' and have _all_ patches reviewed, no matter how trivial. I know it sure reduced my pile of little orange hats over the year, though I still get one once in a while :-). Just ask any senior committer (thats someone who has been around more than 1/2 the life of the project) how many times they could have saved themselves and everyone else some grief had they sent that 10 line patch off for review before commititing it. -- Rod Grimes - KD7CAX @ CN85sl - (RWG25) rgrimes@gndrsh.dnsmgr.net 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?199912192156.NAA38840>