From owner-cvs-all  Thu Dec 20  6:18:36 2001
Delivered-To: cvs-all@freebsd.org
Received: from alcatraz.iptelecom.net.ua (alcatraz.iptelecom.net.ua [212.9.224.15])
	by hub.freebsd.org (Postfix) with ESMTP
	id E3EDE37B416; Thu, 20 Dec 2001 06:18:23 -0800 (PST)
Received: from ipcard.iptcom.net (ipcard.iptcom.net [212.9.224.5])
	by alcatraz.iptelecom.net.ua (8.9.3/8.9.3) with ESMTP id QAA74666;
	Thu, 20 Dec 2001 16:18:04 +0200 (EET)
	(envelope-from sobomax@FreeBSD.org)
Received: from vega.vega.com (h186.228.dialup.iptcom.net [212.9.228.186])
	by ipcard.iptcom.net (8.9.3/8.9.3) with ESMTP id QAA08755;
	Thu, 20 Dec 2001 16:17:56 +0200 (EET)
	(envelope-from sobomax@FreeBSD.org)
Received: from FreeBSD.org (big_brother.vega.com [192.168.1.1])
	by vega.vega.com (8.11.6/8.11.3) with ESMTP id fBKEHNF05117;
	Thu, 20 Dec 2001 16:17:23 +0200 (EET)
	(envelope-from sobomax@FreeBSD.org)
Message-ID: <3C21F2F1.542EE5CC@FreeBSD.org>
Date: Thu, 20 Dec 2001 16:17:21 +0200
From: Maxim Sobolev <sobomax@FreeBSD.org>
Organization: Vega International Capital
X-Mailer: Mozilla 4.79 [en] (Windows NT 5.0; U)
X-Accept-Language: en,uk,ru
MIME-Version: 1.0
To: Bruce Evans <bde@zeta.org.au>
Cc: Peter Wemm <peter@FreeBSD.org>, cvs-committers@FreeBSD.org,
	cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/alpha/alpha pmap.c src/sys/i386/i386 
 pmap.csrc/sys/ia64/ia64 pmap.c
References: <20011221005726.Q884-100000@gamplex.bde.org>
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 7bit
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.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%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

Bruce Evans wrote:
> 
> On Wed, 19 Dec 2001, Peter Wemm wrote:
> 
> > peter       2001/12/19 21:29:59 PST
> >
> >   Modified files:
> >     sys/alpha/alpha      pmap.c
> >     sys/i386/i386        pmap.c
> >     sys/ia64/ia64        pmap.c
> >   Log:
> >   Replace a bunch of:
> >         for (pv = TAILQ_FIRST(&m->md.pv_list);
> >                  pv;
> >                  pv = TAILQ_NEXT(pv, pv_list)) {
> >   with:
> >         TAILQ_FOREACH(pv, &m->md.pv_list, pv_list) {
> >
> >   Revision  Changes    Path
> >   1.79      +9 -28     src/sys/alpha/alpha/pmap.c
> >   1.301     +6 -20     src/sys/i386/i386/pmap.c
> >   1.41      +10 -30    src/sys/ia64/ia64/pmap.c
> 
> This is a regression IMO.  Please write C programs in something
> resembling C.

Why then we have all those *_FOREACH macros in sys/queue.h?

-Maxim

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