Re: cvs commit: src/sys/vm uma_core.c

From: Nate Lawson <nate_at_root.org>
Date: Mon, 9 Jun 2003 15:34:47 -0700 (PDT)
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
> _at__at_ -1843,7 +1843,7 _at__at_
>
>  	/* 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.

-Nate
Received on Mon Jun 09 2003 - 15:34:47 UTC