Date: Wed, 27 May 2009 19:28:04 +0000 (UTC) From: Ed Schouten <ed@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src ObsoleteFiles.inc src/lib/libc/gen popen.c src/share/man/man3 Makefile queue.3 src/sys/sys queue.h Message-ID: <200905271930.n4RJUKV2071468@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
ed 2009-05-27 19:28:04 UTC
FreeBSD src repository
Modified files:
. ObsoleteFiles.inc
lib/libc/gen popen.c
share/man/man3 Makefile queue.3
sys/sys queue.h
Log:
SVN rev 192926 on 2009-05-27 19:28:04Z by ed
Rename the queue macros I introduced last year.
Last year I added SLIST_REMOVE_NEXT and STAILQ_REMOVE_NEXT, to remove
entries behind an element in the list, using O(1) time. I recently
discovered NetBSD also has a similar macro, called SLIST_REMOVE_AFTER.
In my opinion this approach is a lot better:
- It doesn't have the unused first argument of the list pointer. I added
this, mainly because OpenBSD also had it.
- The _AFTER suffix makes a lot more sense, because it is related to
SLIST_INSERT_AFTER. _NEXT is only used to iterate through the list.
The reason why I want to rename this now, is to make sure we don't
release a major version with the badly named macros.
Revision Changes Path
1.193 +3 -0 src/ObsoleteFiles.inc
1.21 +1 -1 src/lib/libc/gen/popen.c
1.50 +2 -2 src/share/man/man3/Makefile
1.43 +20 -20 src/share/man/man3/queue.3
1.72 +5 -5 src/sys/sys/queue.h
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905271930.n4RJUKV2071468>