You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "C. Michael Pilato" <cm...@collab.net> on 2004/02/20 07:50:40 UTC

Re: svn commit: r8713 - trunk/subversion/bindings/swig

breser@tigris.org writes:

> +#ifdef SVN_FILESIZE_T_FMT == "lld"
> +
> +%typemap(python,argout,fragment="t_output_helper") svn_filesize_t *
> +    "$result = t_output_helper($result,
> +                               PyInt_FromLongLong((long long) (*$1)));";
> +

I now get:

   /bin/sh /home/cmpilato/projects/subversion/libtool --silent
   --mode=compile gcc -DNDEBUG -O2 -g -pipe -march=i386 -mcpu=i686
   -D_GNU_SOURCE -fPIC -fPIC -DLINUX=2 -D_REENTRANT -D_XOPEN_SOURCE=500
   -D_BSD_SOURCE -D_SVID_SOURCE -D_GNU_SOURCE
   -I/home/cmpilato/projects/subversion/subversion/bindings/swig
   -I/home/cmpilato/projects/subversion/subversion/include
   -I/usr/local/apache2/include -I/usr/local/apache2/include
   -I/usr/local/BerkeleyDB.4.2/include -I/usr/include/python2.2
   -prefer-pic -c -o subversion/bindings/swig/python/svn_fs.lo
   subversion/bindings/swig/python/svn_fs.c
   subversion/bindings/swig/python/svn_fs.c: In function
   `_wrap_svn_fs_file_length':
   subversion/bindings/swig/python/svn_fs.c:3540: warning: passing arg 2
   of `t_output_helper' makes pointer from integer without a cast

Here's the relevant piece of svn_fs.c:

    ...
    resultobj = t_output_helper(resultobj, 
    PyInt_FromLongLong((long long) (*arg1)));
    ...

And finally, neither Google nor the ActiveState doc search engine
return any hits for PyInt_FromLongLong().  Did you just make that up?
:-)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: r8713 - trunk/subversion/bindings/swig

Posted by Ben Reser <be...@reser.org>.
On Fri, Feb 20, 2004 at 01:50:40AM -0600, C. Michael Pilato wrote:
> breser@tigris.org writes:
> 
> > +#ifdef SVN_FILESIZE_T_FMT == "lld"
> > +
> > +%typemap(python,argout,fragment="t_output_helper") svn_filesize_t *
> > +    "$result = t_output_helper($result,
> > +                               PyInt_FromLongLong((long long) (*$1)));";
> > +
> 
> I now get:
> 
>    /bin/sh /home/cmpilato/projects/subversion/libtool --silent
>    --mode=compile gcc -DNDEBUG -O2 -g -pipe -march=i386 -mcpu=i686
>    -D_GNU_SOURCE -fPIC -fPIC -DLINUX=2 -D_REENTRANT -D_XOPEN_SOURCE=500
>    -D_BSD_SOURCE -D_SVID_SOURCE -D_GNU_SOURCE
>    -I/home/cmpilato/projects/subversion/subversion/bindings/swig
>    -I/home/cmpilato/projects/subversion/subversion/include
>    -I/usr/local/apache2/include -I/usr/local/apache2/include
>    -I/usr/local/BerkeleyDB.4.2/include -I/usr/include/python2.2
>    -prefer-pic -c -o subversion/bindings/swig/python/svn_fs.lo
>    subversion/bindings/swig/python/svn_fs.c
>    subversion/bindings/swig/python/svn_fs.c: In function
>    `_wrap_svn_fs_file_length':
>    subversion/bindings/swig/python/svn_fs.c:3540: warning: passing arg 2
>    of `t_output_helper' makes pointer from integer without a cast
> 
> Here's the relevant piece of svn_fs.c:
> 
>     ...
>     resultobj = t_output_helper(resultobj, 
>     PyInt_FromLongLong((long long) (*arg1)));
>     ...
> 
> And finally, neither Google nor the ActiveState doc search engine
> return any hits for PyInt_FromLongLong().  Did you just make that up?
> :-)

Whoops.  I don't notice the warnings on the python bindings because
there are so many on a 64-bit arch...  All fixed in r8788 and r8791.

-- 
Ben Reser <be...@reser.org>
http://ben.reser.org

"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org