apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aaron Bannert <aa...@clove.org>
Subject Re: [PATCH] add apr_proc_mutex for the new Lock API
Date Fri, 14 Sep 2001 22:23:35 GMT
One note, more of a question to the other APR developers...see my inline
comment:

On Fri, Sep 14, 2001 at 03:15:47PM -0700, Aaron Bannert wrote:
[snip] 
> #if !APR_HAVE_UNION_SEMUN && defined(APR_HAS_SYSVSEM_SERIALIZE)
> /* it makes no sense, but this isn't defined on solaris */
> union semun {
>     long val;
>     struct semid_ds *buf;
>     ushort *array;
> };
> #endif
[snip] 

Since this is now defined in include/arch/unix/proc_mutex.h AND
include/arch/unix/lock.h, if both are #included into a file there
will be redifition errors/warnings. Should we invent a symbol to
protect this section from double-inclusion?

-aaron

Mime
View raw message