[PATCH] ls: clean up memory leak

Rob Landley rob at landley.net
Mon May 1 22:59:15 UTC 2006


On Monday 01 May 2006 6:03 pm, Shaun Jackman wrote:
> > A) So this is dependent on a patch adding a UTF8 config item?
> >
> > B) ICK.  This is the sort of thing platform.h is for, although not having
> > seen the patch I don't know what what exactly would be required.  Some
> > kind of utf8_strlen() macro seems called for, though...
>
> Sorry, Rob. This patch was a conglomeration of a patch by Vladimir N.
> Oleynik <dzo at simtreas.ru> and a patch from myself. I'll clean up the
> patch and remove Vladimir's UTF-8 work and resubmit it.

If it's from Vladimir I'd like to confirm that it's under the GPL, since he's 
explicitly stated that some of his previous patches were not GPL.

And I'm not against UTF-8 support, I'd just like it to be done cleanly, and 
the CONFIG option would need to go in before stuff using that CONFIG option.

> Cheers,
> Shaun

Rob
-- 
Never bet against the cheap plastic solution.



More information about the busybox mailing list