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

Re: svn commit: r1227828 - in /subversion/trunk/tools/examples: SvnCLBrowse get-location-segments.py

From: Daniel Shahaf <danielsh_at_apache.org>
Date: Thu, 5 Jan 2012 23:30:34 +0200

C. Michael Pilato wrote on Thu, Jan 05, 2012 at 16:00:50 -0500:
> On 01/05/2012 03:54 PM, danielsh_at_apache.org wrote:
> > Author: danielsh
> > Date: Thu Jan 5 20:54:09 2012
> > New Revision: 1227828
> >
> > URL: http://svn.apache.org/viewvc?rev=1227828&view=rev
> > Log:
> > Fix the swig-py examples.
> >
> > When trying to run the get-location-segments.py, it throws:
> >
> > TypeError: Unexpected NULL parent pool on proxy object
> >
> > Somehow, the "_parent_pool" property wasn't initialized on the context.
> > Not sure how this works, but creating a context using
> > client.svn_client_create_context() instead of client.ctx_t() (which
> > subversion/bindings/swig/python/tests/pool.py also does) fixes this
> > error and makes the example work again.
>
> I'm not sure what changed since the days when this construct worked, but
> I've seen exactly this symptom (and exactly this solution) before in another
> context[1], so ... +1.
>

In 1.7 the struct has two new members (->wc_ctx and ->conflict_func2)
which the constructor function unconditionally initializes to non-NULL
values.

These are the only members added since 1.5. Virtually all other struct
members are allowed to be NULL. (A few of them are not explicitly
documented as alowing NULL: ->auth_baton, ->notify_func, ->cancel_func.)

</brian dump of potentially relevant observations>

> -- C-Mike
>
> [1] The context should hint that "context" here doesn't refer to the
> problematic context object. ┐Comprendes?
>
> --
> C. Michael Pilato <cmpilato_at_collab.net>
> CollabNet <> www.collab.net <> Distributed Development On Demand
>
Received on 2012-01-05 22:30:56 CET

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