Date: Thu, 04 Jan 2001 10:08:19 -0800 From: Peter Wemm <peter@netplex.com.au> To: Nick Hibma <n_hibma@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern makedevops.pl makeobjops.pl Message-ID: <200101041808.f04I8Jq10348@mobile.wemm.org> In-Reply-To: <200101041341.f04DfPE31302@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Nick Hibma wrote: > n_hibma 2001/01/04 05:41:24 PST > > Modified files: > sys/kern makedevops.pl makeobjops.pl > Log: > Fix a bug in both scripts: HEADER sections were not emitted to the header > file. > > While there fix the layout of function headers (noticable in long headers) > > Fix up some style nits. It's Perl and should be written in that style. > > Revision Changes Path > 1.14 +9 -10 src/sys/kern/makedevops.pl > 1.2 +319 -326 src/sys/kern/makeobjops.pl It is a bug to commit functional changes and reformatting changes in the same commit. Like this, it is practically impossible to see the bug fix you made as it is buried in the noise of the reformatting change. (I dont mean to pick on you specifically, others have been doing this recently too) Cheers, -Peter -- Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au "All of this is for nothing if we don't go to the stars" - JMS/B5 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?200101041808.f04I8Jq10348>