On Wed, Oct 14, 2009 at 05:33:28PM +0100, Stuart Henderson wrote:
> On 2009/10/14 08:38, Matthew Dempsky wrote:
> >
> > Doesn't this leak memory in the -P case? What about moving the
> > xstrdup() call into only the !prefix_builtins path? Then you can also
> > eliminate the const cast:
>
> Yes, Tobias Ulmer noticed this too, I didn't think of a way around
> it before but this makes a lot more sense - thanks.
>
> New diff incorporates your fix and jmc's feedback, and adds a
> simple regression test. (The existing tests still pass).
Looks good. okay.
