From owner-p4-projects  Thu May 23 10: 0:23 2002
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id E117337B403; Thu, 23 May 2002 10:00:15 -0700 (PDT)
Delivered-To: perforce@freebsd.org
Received: from rwcrmhc53.attbi.com (rwcrmhc53.attbi.com [204.127.198.39])
	by hub.freebsd.org (Postfix) with ESMTP
	id 88CB937B40C; Thu, 23 May 2002 10:00:13 -0700 (PDT)
Received: from InterJet.elischer.org ([12.232.206.8])
          by rwcrmhc53.attbi.com
          (InterMail vM.4.01.03.27 201-229-121-127-20010626) with ESMTP
          id <20020523170013.LHQZ11659.rwcrmhc53.attbi.com@InterJet.elischer.org>;
          Thu, 23 May 2002 17:00:13 +0000
Received: from localhost (localhost.elischer.org [127.0.0.1])
	by InterJet.elischer.org (8.9.1a/8.9.1) with ESMTP id JAA88030;
	Thu, 23 May 2002 09:58:02 -0700 (PDT)
Date: Thu, 23 May 2002 09:58:01 -0700 (PDT)
From: Julian Elischer <julian@elischer.org>
To: Jonathan Mini <mini@FreeBSD.org>
Cc: Perforce Change Reviews <perforce@freebsd.org>
Subject: Re: PERFORCE change 11780 for review
In-Reply-To: <200205230506.g4N56Mw18963@freefall.freebsd.org>
Message-ID: <Pine.BSF.4.21.0205230957001.88015-100000@InterJet.elischer.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: owner-p4-projects@FreeBSD.ORG
Precedence: bulk
List-ID: <p4-projects.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20p4-projects>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20p4-projects>
X-Loop: FreeBSD.ORG

I'm a great believer in always having the braces..
but, style(9) wins I guess.


On Wed, 22 May 2002, Jonathan Mini wrote:

> http://people.freebsd.org/~peter/p4db/chv.cgi?CH=11780
> 
> Change 11780 by mini@mini_stylus on 2002/05/22 22:05:58
> 
> 	Remove extraneous braces.
> 
> Affected files ...
> 
> ... //depot/projects/kse/sys/kern/kern_condvar.c#34 edit
> 
> Differences ...
> 
> ==== //depot/projects/kse/sys/kern/kern_condvar.c#34 (text+ko) ====
> 
> @@ -634,11 +634,10 @@
>  	mtx_lock_spin(&sched_lock);
>  	if ((td->td_flags & (TDF_SINTR|TDF_TIMEOUT)) == TDF_SINTR) {
>  		if (td->td_wchan != NULL) {
> -			if (td->td_state == TDS_SLP) {
> +			if (td->td_state == TDS_SLP)
>  				setrunnable(td);
> -			} else {
> +			else
>  				cv_waitq_remove(td);
> -			}
>  		}
>  	}
>  	mtx_unlock_spin(&sched_lock);
> 


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe p4-projects" in the body of the message