On Fri, Jan 9, 2015 at 7:26 PM, Julian Foad <[email protected]>
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.
