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"

Reply via email to