Greg Stein wrote:
> On Sat, Oct 04, 2003 at 01:25:16PM -0500, rey4@tigris.org wrote:
>> Author: rey4
>> Date: Sat Oct 4 13:25:15 2003
>> New Revision: 7296
>>
>> Modified:
>> trunk/subversion/bindings/swig/core.i
>> Log:
>> Make python svn_config_get_config return an opaque pointer to the
>> apr hash of config options instead of converting the hash to a
>> python dictionary. Change motivated by discussion with Barry Scott
>> <barry@barrys-emacs.org> on the user's list.
>
> 1) Development changes should be discussed on dev@, NOT the users
> list. Some of the developers are not subscribed to the users list,
> so they will not be privy to the suggested changes.
>
> 2) After reviewing the thread and the impetus for this change, I
> disagree. The change should be to enable storing a dictionary into
> client_ctx.config rather than adjusting the return value of
> get_config. Python programs have no way to inspect an apr_hash_t
> -- they should *all* be converted to dictionaries on return from
> SWIG, and *all* inputs should allow for a Python dictionary.
>
> I'd ask that this change is reverted, and the logic for ctx.config be
> updated to accept a Python dictionary.
Ok. But what pool should I create the temporary hash in?
- Russ
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 6 21:43:16 2003