Date: Mon, 9 Jun 2003 15:42:47 -0700 (PDT) From: Julian Elischer <julian@elischer.org> To: Nate Lawson <nate@root.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm uma_core.c Message-ID: <Pine.BSF.4.21.0306091541100.46920-100000@InterJet.elischer.org> In-Reply-To: <20030609153323.G36983@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 9 Jun 2003, Nate Lawson wrote: > On Mon, 9 Jun 2003, Poul-Henning Kamp wrote: > > A white-space nit I noticed. > > > > Revision Changes Path > > 1.56 +1 -1 src/sys/vm/uma_core.c > > > > --- src/sys/vm/uma_core.c:1.55 Sun Apr 27 23:11:32 2003 > > +++ src/sys/vm/uma_core.c Mon Jun 9 12:40:34 2003 > > @@ -1843,7 +1843,7 @@ > > > > /* Do we need to remove from any lists? */ > > if (slab->us_freecount+1 == zone->uz_ipers) { > > - LIST_REMOVE(slab, us_link); > > + LIST_REMOVE(slab,us_link); > > LIST_INSERT_HEAD(&zone->uz_free_slab, slab, us_link); > > } else if (slab->us_freecount == 0) { > > LIST_REMOVE(slab, us_link); > > Is that correct style to remove the space after the comma? The former way > seems more correct. Also, it differs from the LIST_REMOVE immediately > after. I am guessing a patch applied backwards or something.. I'm sure that's not what he inteded :-) > > -Nate >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0306091541100.46920-100000>