On Wed, Apr 17, 2013 at 3:21 PM, Bert Huijben <b...@qqmail.nl> wrote: [...] > > Completely unrelated to this patch, but the usage of > both info->props and info->dir->removed_props looks incorrect. > > Both props and removed_props are documented to be stored in > the directory, but while removed_props is used that way the props > hash sometimes is and sometimes isn't. > > Is the documentation partially outdated, or are we just 'lucky' that > we don't test property removals that often in our testsuite? > I'm not familiar with pool usage in ra_serf update editor. Most likely we are just 'lucky' because pool handling extremely complicated. So you can just fix pool usage if you think current implementation has problems.
-- Ivan Zhakov CTO | VisualSVN | http://www.visualsvn.com
