On Fri, 15 Jul 2011 01:36:05 +1000, Benjamin Herrenschmidt <[email protected]> wrote: > On Thu, 2011-07-14 at 07:57 -0700, Dan Nicholson wrote: > > I also see a couple instances of "free (val.str)" in parser/Files.c > > that don't set it to NULL afterward. Yay for custom parsers! With > > those two instances fixed: > > Hrm, did I have a grep failure ? Oh... I see, I didn't catch the space > between free and (val.str). I can send a newer patch tomorrow, or an > addon, let me know, I'm off to bed now.
Might as well send a single patch which fixes all of them in one go. I also wouldn't mind seeing a patch to replace 'val' with a slightly more appropriate name :-) -- [email protected]
pgpV9JzZ9MfI1.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
