Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Jun 2009 13:03:01 +0300
From:      Andriy Gapon <avg@freebsd.org>
To:        Brooks Davis <brooks@freebsd.org>
Cc:        svn-src-projects@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r194389 - projects/ngroups/sys/kern
Message-ID:  <4A3A10D5.8000601@freebsd.org>
In-Reply-To: <200906171940.n5HJerMr086037@svn.freebsd.org>
References:  <200906171940.n5HJerMr086037@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
on 17/06/2009 22:40 Brooks Davis said the following:
> Author: brooks
> Date: Wed Jun 17 19:40:53 2009
> New Revision: 194389
> URL: http://svn.freebsd.org/changeset/base/194389
> 
> Log:
>   use MIN() instead of min() in one case.

I think 'what' was pretty obvious from the diff, 'why' is what I am curious about :-)
Just trying to learn something.

> Modified:
>   projects/ngroups/sys/kern/kern_prot.c
> 
> Modified: projects/ngroups/sys/kern/kern_prot.c
> ==============================================================================
> --- projects/ngroups/sys/kern/kern_prot.c	Wed Jun 17 19:01:47 2009	(r194388)
> +++ projects/ngroups/sys/kern/kern_prot.c	Wed Jun 17 19:40:53 2009	(r194389)
> @@ -1899,7 +1899,7 @@ cru2x(struct ucred *cr, struct xucred *x
>  	xcr->cr_version = XUCRED_VERSION;
>  	xcr->cr_uid = cr->cr_uid;
>  
> -	ngroups = min(cr->cr_ngroups, XU_NGROUPS);
> +	ngroups = MIN(cr->cr_ngroups, XU_NGROUPS);
>  	xcr->cr_ngroups = ngroups;
>  	bcopy(cr->cr_groups, xcr->cr_groups,
>  	    ngroups * sizeof(*cr->cr_groups));


-- 
Andriy Gapon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4A3A10D5.8000601>