Re: Bug in svn_stringbuf_insert and svn_stringbuf_replace

On Fri, Jan 9, 2015 at 7:26 PM, Julian Foad <julianfoad_at_btopenworld.com>
wrote:

> Stefan Fuhrmann wrote:
> [...]
> > Yes, you are right. I'll fix this.
> [...]
> > I'll add test code for these as well.
>
> Thanks.
>

Done in r1650834.

After that you might want to eliminate the recursion, trivially, in both
> functions:
>
> if //it overlaps
> {
> bytes = apr_pmemdup(str->pool, bytes, count);
> // don't recurse here
>
> }
> //continue here with the rest of the function
>

Thanks, that is much better.
Implemented in r1650835.

-- Stefan^2.
Received on 2015-01-14 01:51:23 CET