From owner-cvs-sys Fri Mar 14 09:19:33 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.5/8.8.5) id JAA29515 for cvs-sys-outgoing; Fri, 14 Mar 1997 09:19:33 -0800 (PST) Received: from who.cdrom.com (who.cdrom.com [204.216.27.3]) by freefall.freebsd.org (8.8.5/8.8.5) with ESMTP id JAA29432; Fri, 14 Mar 1997 09:17:47 -0800 (PST) Received: from nic.follonett.no (nic.follonett.no [194.198.43.10]) by who.cdrom.com (8.8.5/8.6.11) with ESMTP id JAA17756 ; Fri, 14 Mar 1997 09:17:31 -0800 (PST) Received: (from uucp@localhost) by nic.follonett.no (8.8.5/8.8.3) with UUCP id SAA07682; Fri, 14 Mar 1997 18:14:11 +0100 (MET) Received: from oo7 (oo7.dimaga.com [192.0.0.65]) by dimaga.com (8.7.5/8.7.2) with SMTP id SAA02097; Fri, 14 Mar 1997 18:20:25 +0100 (MET) Message-Id: <3.0.32.19970314181837.0163ba10@dimaga.com> X-Sender: eivind@dimaga.com X-Mailer: Windows Eudora Pro Version 3.0 (32) Date: Fri, 14 Mar 1997 18:18:38 +0100 To: Nate Williams From: Eivind Eklund Subject: Re: cvs commit: src/sys/i386/include endian.h Cc: CVS-committers@freefall.freebsd.org, cvs-all@freefall.freebsd.org, cvs-sys@freefall.freebsd.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: owner-cvs-sys@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk At 08:19 AM 3/14/97 -0800, Nate Williams wrote: >nate 97/03/14 08:19:21 > > Branch: sys/i386/include RELENG_2_1_0 > Modified: sys/i386/include endian.h > Log: > YAMFC: > > When compiling with '-Wall -pedantic', the byte swapping macros gave the > following warning: > warning: ANSI C forbids braced-groups within expressions > > Adding __extension__ before the statement-expression seems to work right. While you're in there, why not reverse the sense of the LINT flag? As it is, linting get the GCC extensions, and a page of errors. Besides, NTOHL etc are missing either parentheses or do{...}while(0) - preferably the last. Presently, they get precedence errors if used in expressions. Eivind Eklund perhaps@yes.no http://maybe.yes.no/perhaps/ eivind@freebsd.org