On Fri, Apr 06, 2007 at 11:58:55AM +1000, Bruce Evans wrote:
> On Thu, 5 Apr 2007, Pawel Jakub Dawidek wrote:
> 
> >pjd         2007-04-05 21:10:54 UTC
> >
> > FreeBSD src repository
> >
> > Modified files:
> >   include              stdio.h
> >   lib/libc/sys         lseek.2 pathconf.2
> >   sys/kern             vfs_syscalls.c
> >   sys/sys              filio.h unistd.h
> > Log:
> > Implement SEEK_DATA and SEEK_HOLE extensions to lseek(2) as found in
> > OpenSolaris. For more information please refer to:
> >
> >         http://blogs.sun.com/bonwick/entry/seek_hole_and_seek_data
> 
> These are just namespace pollution in stdio.h.  stdio (fseek()) doesn't
> support them, and the hack of declaring lseek() in stdio.h should not
> be extended to new interfaces.  If stdio ever supports them, then they
> would need to be declared under a visibility ifdef (not #ifdef self).

Ok, I removed SEEK_DATA/SEEK_HOLE from stdio.h and hidden them in
sys/unistd.h under:

#if !defined(_POSIX_SOURCE) && !defined(_XOPEN_SOURCE)
[...]
#endif

This is consistent which what Solaris has.

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
[EMAIL PROTECTED]                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!

Attachment: pgpn6nAVLt70F.pgp
Description: PGP signature

Reply via email to