On Fri, Jan 9, 2015 at 7:26 PM, Julian Foad <julianf...@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.

Reply via email to