cvs commit: src/share/man/man3 Makefile queue.3 src/sys/sys queue.h

Maxim Sobolev sobomax at FreeBSD.org
Thu May 22 17:16:38 UTC 2008


Sam Leffler wrote:
> Maxim Sobolev wrote:
>> Ed Schouten wrote:
>>>   The OpenBSD version of SLIST_REMOVE_NEXT() needs a reference to the 
>>> list
>>>   head, even though it is unused. 
>>
>> I don't think it's necessary. sys/queue.h API is not supposed to be 
>> portable across different operating systems. The need to pass list 
>> head around may impede macro usefulness.
> Gratuitous differences impede porting.  If it's not a big deal then 
> maintain compatibility.  We already have stupid differences in various 
> areas that must be papered over when code is brokered between systems.

This is local use convenience vs. easy of porting tradeoff. On one hand 
API is used heavily in the kernel on the other I don't think we have lot 
of kernel code being ported from OpenBSD nowadays. So that local use 
convinience should win a big.

I could be wrong, though.

-Maxim


More information about the cvs-src mailing list