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

RE: Crash in python bindings on windows (in apache)

From: Marc Haesen <Marc.Haesen_at_telindus.be>
Date: 2005-10-20 08:52:04 CEST

You are right. The error was there because there was a statement before a variable declaration, not the (void)pool itself.

Regards,
Marc

-----Original Message-----
From: David James [mailto:james82@gmail.com]
Sent: Wednesday, October 19, 2005 21:39
To: Branko Cibej
Cc: Marc Haesen; Max Bowsher; dev@subversion.tigris.org; Daniel L. Rall; David Anderson
Subject: Re: Crash in python bindings on windows (in apache)

On 10/19/05, David James <james82@gmail.com> wrote:
> On 10/19/05, Branko Čibej <brane@xbc.nu> wrote:
> > David James wrote:
> >
> > >On 10/19/05, Marc Haesen <Marc.Haesen@telindus.be> wrote:
> > >
> > >
> > >>I have just updated my working copy to revision 16815 and after applying
> > >>the attached patch (to solve a compilation problem in windows), it seems
> > >>not to crash anymore in apache with mod_python.
> > >>
> > >>In the patch.txt I have just removed the line (void) pool;
> > >>It can also replaced by pool=pool; to avoid a warning.
> > >>
> > >>
> > >This is fantastic news, Marc. I have followed your suggested approach,
> > >using pool = pool, to avoid a warning in r16817. Thanks so much for
> > >your help and your patience!
> > >
> > >
> > I'd really like to know what sort of Windows compile problem the "(void)
> > pool" thing caused. This is an idiom we've been using throughout the
> > code to silence warnings about unused variables. Changing this single
> > isntance is nonsense, especially since the "compile problem" report
> > wasn't exactly enlightening.
>
> It does seem strange that "(void) pool;" would not work on Windows,
> especially considering how often we use the same idiom elsewhere in
> the code:
>
> subversion/libsvn_subr/config.c: (void)(baton); /*
> Unused parameter. */
> subversion/libsvn_subr/config.c: (void)(section); /*
> Unused parameter. */
> subversion/libsvn_wc/adm_files.c: (void)pool; /* Silence compiler
> warnings about unused parameter. */
> subversion/libsvn_wc/adm_files.c: (void)pool; /* Silence compiler
> warnings about unused parameter. */
>
> Marc, can you take a look at this?

An update: It looks like MSVC is fine at handling the "(void) pool"
idiom, but was being tripped up by the fact that we had statements
before declarations. I've reverted r16817 and committed a better fix
in r16842. Marc, can you confirm that this new fix solves the problem?

Cheers,

David

--
David James -- http://www.cs.toronto.edu/~james
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 20 08:53:04 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.