[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Subversion 1.3.0 RC1

From: Kouhei Sutou <kou_at_cozmixng.org>
Date: 2005-10-23 04:34:09 CEST

Hi,

In <435AF0E3.5060905@calixo.net>
  "Re: Subversion 1.3.0 RC1" on Sun, 23 Oct 2005 04:09:39 +0200,
  David Anderson <david.anderson@calixo.net> wrote:

> It is the only solution. We cannot import apr.h in the SWIG interface
> code, because otherwise autogen.sh --release breaks completely.

I see.

> However, I'm not sure that mapping apr_uint32_t to unsigned int will
> work consistently on 32 and 64 bit platforms. Intuitively, I reckon
> this breaks and needs to be defined more carefully, so as to map to the
> right types on both 32 and 64 bit architectures.

I checked apr.h on FreeBSD/i386 (32 bit) and FreeBSD/amd64
(64 bit) platforms. apr_uint32_t is typemapped as unsigned
int on both platforms.

> Thoughts?

Ruby bindings need more %applys.

  %apply unsigned int *OUTPUT { apr_uint32_t * };
  %apply size_t { apr_size_t };
  %apply size_t *OUTPUT { apr_size_t * };

Thanks,

--
kou

Index: subversion/bindings/swig/include/apr.swg
===================================================================
--- subversion/bindings/swig/include/apr.swg (revision 16922)
+++ subversion/bindings/swig/include/apr.swg (working copy)
@@ -48,6 +48,12 @@
 
 #undef OUTPUT_TYPEMAP
 #endif
+
+%apply unsigned int { apr_uint32_t };
+%apply unsigned int *OUTPUT { apr_uint32_t * };
+%apply size_t { apr_size_t };
+%apply size_t *OUTPUT { apr_size_t * };
+
 #endif
 
 /* ----------------------------------------------------------------------- */

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Oct 23 04:34:56 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.