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

Re: [PATCH] fix unable to load *.pyd files with Python 3.8.x on Windows

From: Yasuhito FUTATSUKI <futatuki_at_poem.co.jp>
Date: Sat, 23 May 2020 21:15:56 +0900

On 2020/05/23 12:16, Daniel Shahaf wrote:
> Yasuhito FUTATSUKI wrote on Sat, 23 May 2020 11:08 +0900:
>> I think it can be fixed by overriding _global_py_pool to $input in "in"
>> typemap, when $1 is updated. It assumes that there are no APIs that
>> uses two or more result_pool like apr_pool_t * argument for allocationg
>> return value.
>
> What about svn_repos_node_editor()? It has two pools, and if I'm
> reading the docstring correctly, "pool" will be used both for temporary
> allocations and for allocating *editor and *edit_baton, while
> "node_pool" will be used to allocate the tree that function constructs.
>
> That function is from 1.0 [according to its docstring], so it precedes
> the result_pool/scratch_pool convention by several years.

We can't add reference of "node_pool" Python object in apr_pool_t *
members of the *editor and the *edit_baton C structure, at least
with current wrapper mechanism. I think all we can do is to make
"_parent_pool" attribute of wrapper objects for *editor and
*edit_baton point to "pool" object.

Fortunately as apr_pool_t *node_pool is used this API only, we can
add typemap for it safely, in this case.

Cheers,

-- 
Yasuhito FUTATSUKI <futatuki_at_poem.co.jp>/<futatuki_at_yf.bsdclub.org>

Received on 2020-05-23 14:17:04 CEST

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