From owner-cvs-src-old@FreeBSD.ORG Wed May 27 19:30:21 2009 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0DFB31065700 for ; Wed, 27 May 2009 19:30:21 +0000 (UTC) (envelope-from ed@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id EE8B98FC1A for ; Wed, 27 May 2009 19:30:20 +0000 (UTC) (envelope-from ed@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n4RJUK5D071469 for ; Wed, 27 May 2009 19:30:20 GMT (envelope-from ed@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n4RJUKV2071468 for cvs-src-old@freebsd.org; Wed, 27 May 2009 19:30:20 GMT (envelope-from ed@repoman.freebsd.org) Message-Id: <200905271930.n4RJUKV2071468@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to ed@repoman.freebsd.org using -f From: Ed Schouten Date: Wed, 27 May 2009 19:28:04 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src ObsoleteFiles.inc src/lib/libc/gen popen.c src/share/man/man3 Makefile queue.3 src/sys/sys queue.h X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 27 May 2009 19:30:21 -0000 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