apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)" <madhusudan_mathiha...@hp.com>
Subject RE: [PATCH] Regarding semun definition in proc_mutex.h
Date Thu, 27 Jun 2002 17:31:25 GMT
'just wondering if somebody could spend 5 seconds to give a thought to it
:-).

-Madhu

-----Original Message-----
From: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
[mailto:madhusudan_mathihalli@hp.com]
Sent: Wednesday, June 26, 2002 9:35 AM
To: 'dev@apr.apache.org'
Subject: [PATCH] Regarding semun definition in proc_mutex.h


On platforms where the semun union is not available, the APR defines the
semun structure - but I believe the first parameter should be a "int val"
and not "long val". I verified it on HP-UX and Linux (both 32-bit and 64-bit
systems).

BTW, we were able to get Apache (2.0.39) to work on the HP-UX / Itanium
architecture without any other problems. Great work !!.

-Madhu

Index: proc_mutex.h
===================================================================
RCS file: /home/cvspublic/apr/include/arch/unix/proc_mutex.h,v
retrieving revision 1.8
diff -u -r1.8 proc_mutex.h
--- proc_mutex.h        7 Jun 2002 14:04:34 -0000       1.8
+++ proc_mutex.h        26 Jun 2002 16:29:52 -0000
@@ -142,7 +142,7 @@
 
 #if !APR_HAVE_UNION_SEMUN && defined(APR_HAS_SYSVSEM_SERIALIZE)
 union semun {
-    long val;
+    int val;
     struct semid_ds *buf;
     unsigned short *array;
 };

Mime
View raw message