Date: Thu, 10 Jan 2008 08:59:59 +0000 (GMT) From: Robert Watson <rwatson@FreeBSD.org> To: Sam Leffler <sam@errno.com> Cc: src-committers@FreeBSD.org, cvs-all@FreeBSD.org, obrien@FreeBSD.org, cvs-src@FreeBSD.org, Max Laier <max@love2party.net> Subject: Re: cvs commit: src/sys/netinet6 frag6.c icmp6.c in6.c in6_ifattach.c in6_pcb.c in6_proto.c in6_rmx.c in6_src.c ip6_input.c ip6_mroute.c ip6_output.c mld6.c nd6.c nd6_nbr.c nd6_rtr.c raw_ip6.c udp6_usrreq.c Message-ID: <20080110085803.H66908@fledge.watson.org> In-Reply-To: <478531AC.5030303@errno.com> References: <200801081908.m08J8wIs094059@repoman.freebsd.org> <200801082039.18671.max@love2party.net> <e7db6d980801082231p409d86bp34127fa153b56d7a@mail.gmail.com> <20080109170850.GA27993@dragon.NUXI.org> <4785058B.1050507@errno.com> <20080109201424.GA4969@server.vk2pj.dyndns.org> <478531AC.5030303@errno.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 9 Jan 2008, Sam Leffler wrote: > Peter Jeremy wrote: >> On Wed, Jan 09, 2008 at 09:34:03AM -0800, Sam Leffler wrote: >> >>> I observe that netbsd does these sorts of changes in sweeps over the tree >>> and w/ multiple people involved to review each other's work. I'd be more >>> confident if I saw commits like this include a "Reviewed by:" line. >> >> For this sort of commit, it should also be possible to check that you >> haven't broken anything by cmp(1)ing the .o files or manually preprocessing >> and diffing the preprocessed files. > > For this specific case md5 checksums of the resulting .o files should work. > However some committers have a history of making changes like this that > either break the build (w/o a quick followup to correct the problem) or > introduce subtle issues that aren't found until much later. Hence my > request (removed by obrien) that people do changes like this when they are > actively working on a section of code. Drive-by commits like this > frequently do more harm than good. In the past, we've had several subtle bugs crop up as a result of changing to ANSI C function prototypes, which no longer automatically promote arguments to int from shorter types (I think it's that way around). This caused some very subtle semantic changes in the socket code a few years ago and was a nightmare to track down. Unfortunately, I don't remember the details or I'd dig up the commit that fixed it, it was really quite tricky, as what on face value was a style change actually changed the semantics significantly. Robert N M Watson Computer Laboratory University of Cambridge
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080110085803.H66908>