From owner-cvs-all@FreeBSD.ORG Thu Jan 10 09:00:00 2008 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 924F016A417; Thu, 10 Jan 2008 09:00:00 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [209.31.154.42]) by mx1.freebsd.org (Postfix) with ESMTP id 33C0113C457; Thu, 10 Jan 2008 09:00:00 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [209.31.154.41]) by cyrus.watson.org (Postfix) with ESMTP id B2BCE47278; Thu, 10 Jan 2008 03:59:59 -0500 (EST) Date: Thu, 10 Jan 2008 08:59:59 +0000 (GMT) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Sam Leffler In-Reply-To: <478531AC.5030303@errno.com> Message-ID: <20080110085803.H66908@fledge.watson.org> References: <200801081908.m08J8wIs094059@repoman.freebsd.org> <200801082039.18671.max@love2party.net> <20080109170850.GA27993@dragon.NUXI.org> <4785058B.1050507@errno.com> <20080109201424.GA4969@server.vk2pj.dyndns.org> <478531AC.5030303@errno.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: src-committers@FreeBSD.org, cvs-all@FreeBSD.org, obrien@FreeBSD.org, cvs-src@FreeBSD.org, Max Laier 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 X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 10 Jan 2008 09:00:00 -0000 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