Date: Sun, 05 Dec 1999 14:14:17 +0800 From: Peter Wemm <peter@netplex.com.au> To: Marcel Moolenaar <marcel@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/linux linux_ioctl.h imgact_linux.c linux.h linux_ioctl.c linux_sysvec.c linux_util.h Message-ID: <19991205061417.901071CA0@overcee.netplex.com.au> In-Reply-To: Message from Marcel Moolenaar <marcel@FreeBSD.org> of "Sat, 04 Dec 1999 03:10:23 PST." <199912041110.DAA62066@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Marcel Moolenaar wrote: > Log: > Implement pluggable ioctl handlers. [..] > linux.h and linux_ioctl.[ch] have been made to conform to style(9) as > much as possible. [..] > Revision Changes Path > 1.35 +3 -1 src/sys/i386/linux/imgact_linux.c > 1.41 +302 -650 src/sys/i386/linux/linux.h > 1.48 +1343 -1127src/sys/i386/linux/linux_ioctl.c [..] This is a classic example of why *NOT* to do this. The actual change is lost in the noise. It would have been far better to do this as two seperate commits, as it's quite useless like this. Cheers, -Peter 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?19991205061417.901071CA0>