Re: svn commit: r1468439 - /subversion/trunk/subversion/libsvn_ra_serf/update.c
On Wed, Apr 17, 2013 at 3:21 PM, Bert Huijben <bert_at_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
CTO | VisualSVN | http://www.visualsvn.com
Received on 2013-04-17 14:00:22 CEST
This is an archived mail posted to the Subversion Dev