Stefan Sperling <s...@elego.de> writes: > On Wed, Apr 03, 2013 at 02:58:56PM +0400, Ivan Zhakov wrote: >> I think result_pool parameter should be replaced with pool member in >> svn_repos_t structure. > > Or, as Philip suggested, we could replace the svn_repos_hooks_setenv() > function with a new 'hooks_env_path' parameter to svn_repos_open().
With a pool in the structure we could cache the lazily created hash in the structure as well. Then when running multiple hooks, pre- and post- commit or pre- and post- unlock say, we would only parse the environment file once. -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download
