That function performs a shallow copy. It doesn't know how to duplicate
the referenced values or keys. (And the config is inside something like
a multilevel hash)

Bert Huijben (Cell phone)
From: Stefan Küng
Sent: 4-9-2012 19:33
To: dev@subversion.apache.org
Subject: Re: thread safe
On 03.09.2012 22:21, Branko Čibej wrote:
> On 03.09.2012 22:16, Stefan Küng wrote:
>> On 03.09.2012 22:11, Branko Čibej wrote:
>>> Either that, or add a new API that creates a deep-copy of the in-memory
>>> svn_config_t structure, making this another way to avoid re-reading the
>>> config file for each thread and avoiding the hassle of managing a mutex.
>>
>> I'm in favor of implementing a deep copy API for this. I think that
>> would be the best solution.
>
> I find it to be the best option, too. Will you write a patch for that?

Ahem:
apr_hash_copy
will do the trick: svn_config_t is a typedef for an apr_hash_t.

So it's already done :)

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net

Reply via email to