On May 21, 2012, at 8:57 AM, Hartmut Brandt wrote:
> it seems that there is a syntax error in malloc_np.h. It contains lines like:
> 
> int   allocm(void **ptr, size_t *rsize, size_t size, int flags)
>    __attribute__(nonnull(1));
> 
> The problem is that the syntax of the attributes actually requires two
> parenthesis:
> 
>    __attribute__((nonnull(1)));
> 
> Our gcc seems to swallow either of both, but gcc47 chokes on the missing
> '(' as does clang++.
> 
> Given that we have __nonnull(x) in cdefs.h I suppose the above is better
> replaced with __nonnull(1)?

__nonnull(1) looks like the best fix to me.  Let me know if you want me to make 
the fix, otherwise I'll leave it to you.

Thanks,
Jason_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to