From owner-freebsd-hackers  Sat Jul 26 01:21:41 1997
Return-Path: <owner-freebsd-hackers>
Received: (from root@localhost)
          by hub.freebsd.org (8.8.5/8.8.5) id BAA20407
          for hackers-outgoing; Sat, 26 Jul 1997 01:21:41 -0700 (PDT)
Received: from helbig.informatik.ba-stuttgart.de (rvc1.informatik.ba-stuttgart.de [141.31.112.22])
          by hub.freebsd.org (8.8.5/8.8.5) with ESMTP id BAA20394
          for <freebsd-hackers@FreeBSD.ORG>; Sat, 26 Jul 1997 01:21:34 -0700 (PDT)
Received: (from helbig@localhost)
	by helbig.informatik.ba-stuttgart.de (8.8.6/8.8.5) id KAA10500;
	Sat, 26 Jul 1997 10:11:46 +0200 (MET DST)
From: Wolfgang Helbig <helbig@MX.BA-Stuttgart.De>
Message-Id: <199707260811.KAA10500@helbig.informatik.ba-stuttgart.de>
Subject: Re: Hello boys its me again.
In-Reply-To: <Pine.SOL.3.91.970725161441.2931A-100000@artemis> from Robin Carey at "Jul 25, 97 04:20:21 pm"
To: r.carey@dcs.napier.ac.uk (Robin Carey)
Date: Sat, 26 Jul 1997 10:11:43 +0200 (MET DST)
Cc: freebsd-hackers@FreeBSD.ORG
X-Mailer: ELM [version 2.4ME+ PL30 (25)]
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: owner-freebsd-hackers@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

Robin Carey writes:

> Sorry for not send-pr'ing this ... I noticed this the other day:
> 
> Basically you've got bzero(3) and bcopy(3) being used in
> /usr/include/sys/types.h for the FD_ZERO() and FD_SET() macros, and you've
> not included <stdlib.h> - or is it <string.h> - never can tell with these
> dodgy non-ANSI calls :) This results in a warning with gcc -Wall if you

The FD_???? macros support the select(2) call.  Both the bzero(3)
and select(2) calls belong to the same compatibility class:  Neither
ANSI nor POSIX but BSD (and supported by other Unices that emulate
(part of) BSD). There is no harm if one depends on the other and
its desirable to provoke a warning if compiled with -ansi and -Wall.
This warning would not be given if memset(3) is used instead.

> don't accidentally include the right header before <sys/types.h> .... I'd
> suggest changing them to memset(3) and memcpy(3) respectively, which are 
> ANSI and need <string.h>. BTW Are those FD_*() macros meant to be in 
> <sys/types.h> ? I'd have thought <unistd.h> would be a better place, but 
> thats just my warped sense of direction :)

Your suggested change is gratituos and would break BSD compatibility.

Instead we should add the ``#include <string.h>'' clause to all
man pages that have the ``#include <sys_types.h>'' line or make
sys/types.h self sufficient to satisfy -Wall.

Wolfgang

PS
cc changed from -bugs to -hackers.

> 
> L8r.
>