Date: Tue, 23 Jan 2001 22:04:34 +1100 (EST) From: Bruce Evans <bde@zeta.org.au> To: Matthew Jacob <mjacob@feral.com> Cc: Matt Jacob <mjacob@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_synch.c Message-ID: <Pine.BSF.4.21.0101232158240.36707-100000@besplex.bde.org> In-Reply-To: <Pine.BSF.4.21.0101230038110.4435-100000@beppo.feral.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 23 Jan 2001, Matthew Jacob wrote: > > On Tue, 23 Jan 2001, Bruce Evans wrote: > > > On Mon, 22 Jan 2001, Matt Jacob wrote: > > > > > mjacob 2001/01/22 14:22:39 PST > > > > > > Modified files: > > > sys/kern kern_synch.c > > > Log: > > > Move (now) unused variable declaration inside the block (now commented out). > > > > This actually moves the (temporarily) unused declaration to /dev/null. > > Please ifdef temporary changes. Don't move them to a wrong place. > > Not sure I follow. I rather think this was the right place to put it. The > declaration was for a value only used within a (now #if 0'd ) block. I suppose > I could have nuked the whole block too. Principle of minimal messes. The code is correctly placed where it was since it is not going away. Bruce 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?Pine.BSF.4.21.0101232158240.36707-100000>