Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 17 May 2001 21:11:21 +0100
From:      Brian Somers <brian@Awfulhak.org>
To:        Peter Wemm <peter@wemm.org>
Cc:        Ruslan Ermilov <ru@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, brian@Awfulhak.org
Subject:   Re: cvs commit: src/sys/sys sysctl.h 
Message-ID:  <200105172011.f4HKBLb68306@hak.lan.Awfulhak.org>
In-Reply-To: Message from Peter Wemm <peter@wemm.org>  of "Thu, 17 May 2001 12:22:46 PDT." <20010517192246.99AA8380A@overcee.netplex.com.au> 

next in thread | previous in thread | raw e-mail | index | archive | help
> Ruslan Ermilov wrote:
> > ru          2001/05/17 10:57:37 PDT
> > 
> >   Modified files:
> >     sys/sys              sysctl.h 
> >   Log:
> >   Cosmetics to rev.1.89: removed argument names in a function prototype.
> 
> It is also correct to merely move the names out of application
> space.  ie, change:
> int foo(int *bar);
> to:
> int foo(int *_bar);
> 
> For the bystanders, this is so that if somebody already has a macro:
> #define bar 1
> then a later #include <sys/sysctl.h> will fail due to: int foo(int *1);
> Applications are perfectly entitled to do this.

On a somewhat more esoteric note, when compiling with -Wshadow and 
using something like

	int foo(int link)

you end up with a warning if you include unistd.h (it confuses 
``link'' with the function prototype.

This may be a compiler bug though :)

> Cheers,
> -Peter
> --
> Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au
> "All of this is for nothing if we don't go to the stars" - JMS/B5

-- 
Brian <brian@Awfulhak.org>                        <brian@[uk.]FreeBSD.org>
      <http://www.Awfulhak.org>;                   <brian@[uk.]OpenBSD.org>
Don't _EVER_ lose your sense of humour !



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




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