Re: svn commit: r1664078 - in /subversion/trunk/subversion: include/private/ libsvn_ra/ libsvn_ra_local/ libsvn_ra_serf/ libsvn_ra_svn/ libsvn_subr/

[Philip Martin]

Philip Martin <philip.mar...@wandisco.com> writes:

>>    if (err)
>>      {
>> +      svn_pool_clear(scratch_pool);
>
> As this path always returns an error I think it would be clearer to
> destroy the temporary pool rather than clear it.  I think it should be
> the parent sesspool rather than scratch_pool, as in the other error
> paths.

I've changed this in r1664344.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*