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