At 1245411976 time_t, Uli Schlachter wrote: > today I took a look at valgrind's 'leak-check=full' output and thought "wtf? I > gotta improve this!". As a first step I added a call to lua_close() in > awesome_atexit() which (finally?) frees the complete lua state during cleanup. > > The downside of this is that awesome starts to crash during shutdown. :( > I had some fun with the example config, traced this down to awful.menu.new(), > had some more fun and came up with the attached test case which is quite > minimal > (yes, this is supposed to be a "complete" config). > > Attached are the stripped-down test case and a patch for awesome.c which adds > the lua_close() call. I hope you can help me with finding the crash bug.
Seems I managed to make the bug disappear in next after reworking all the suff you pointed as broken, you bastard. :) I think this is more a sight effect than a real fix, since there's still a couple of things that are not handled properly, like button and keys arrays. I do not think I'm going to fix that as it is, I'll rather try to add my new container class and fix that in the same round. Testing welcome anyway. Cheers, -- Julien Danjou // ᐰ <[email protected]> http://julien.danjou.info // 9A0D 5FD9 EB42 22F6 8974 C95C A462 B51E C2FE E5CD // Tomorrow I was nothing, yesterday I'll be.
signature.asc
Description: Digital signature
