On Sat, 19 May 2012, Konstantin Belousov wrote:
Log:
Hide DIR definition by making it an opaque struct typedef.
Introduce dirfd() libc exported symbol replacing macro with same name,
preserve _dirfd() macro for internal use.
Replace dirp->dd_fd with dirfd() call. Avoid using dirfd as variable
name to prevent shadowing global symbol.
Sponsored by: Google Summer Of Code 2011
Added:
head/lib/libc/gen/dirfd.c (contents, props changed)
head/lib/libc/gen/gen-private.h (contents, props changed)
I could agree with attribution of the gen-private.h content to 'regents
of the university' in the copyright, but I very much doubt that this
is right thing to do for dirfd.c.
I think you should put your name, instead of university, as the copyright
owner.
gen-private.h is also missing all history and in particular is missing
the history needed to connect it to the regents.
gen-private.h also has a gnu-style name. If it had been provided by
the regents or their direct contributors, then it would have a BSD-style
name (gen_private.h or just genprivate.h, or maybe gen.h or dirfd.h
(headers in libc/gen are private by definition; if they were private
to libc and not just libc/gen, then they would be in libc/include)).
In 4.4BSD-Lite2, there are 22246 files; 1237 of these are named *-*,
but most of these are in src/contrib and other contribed files (mainly
usr.sbin/amd); there are about 20 files whose name has a hyphen and
whose contents has a regents copyright; none of these are in src/lib
but 5 are in src/sys. In FreeBSD now, the naming scheme has already
rotted to about 55 names with a hyphen in src/lib and 524 in src/sys
(more than 1/2 of the latter in contrib/octeon and almost 1/4 in
contribed drivers).
Bruce
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"