Re: CVS commit: src/common/lib/libc/string

[Matt Thomas <matt%3am-software.com@localhost>]

On Mar 2, 2012, at 11:15 AM, Alan Barrett wrote:

> On Fri, 02 Mar 2012, Jukka Ruohonen wrote:
>>> Log Message:
>>> Change CTASSERT to __CTASSERT (the spelling used in <sys/cdefs.h>),
>>> include <sys/cdefs.h> so that it will be defined,
>>> and move it to a better place.
>>> 
>>> The previous CTASSERT would never have been used because
>>> of the ifdef.  If it had been used, it would have had
>>> unwanted effects from being just after the "static inline"
>>> that appears when BZERO is defined.
>>> 
>>> Also move the __RCSID to a more conventional location.
>> 
>> And what was the resolution with all this? (cf. useless changes?).
> 
> Sorry, I don't understand what you mean.
> 
>> Please change the documentation too (and most of all, !).
> 
> It seems taht CTASSERT is defined in libkern.h, so is available to
> kernel code.  I'll fix the documentation.

__CTASSERT is available everywhere