On Thu, Jan 29, 2009 at 11:43:36AM +0000, Jeremy Henty wrote:
On Wed, Jan 28, 2009 at 10:44:11PM +0100, Hofmann Johannes wrote:
On Wed, Jan 28, 2009 at 01:06:51PM +0000, Jeremy Henty wrote:
Went with the former for now, but we will need to switch to the latter to support "inherit" or properties that can hold strings _or_ enums. Please check current tip.
I still see a leak. I'm attaching a test case, the valgrind log and a suppression that stops the report.
Not sure about those. Maybe it's reporting the strings allocated for the user-agent style? Those are allocated once and never freed. Can you please retry with the user-agent style parsing disabled: diff -r 4b5125e3ca89 src/css.cc --- a/src/css.cc Thu Jan 29 12:38:20 2009 +0100 +++ b/src/css.cc Thu Jan 29 13:24:16 2009 +0100 @@ -327,7 +327,7 @@ CssContext::CssContext () { sheet[CSS_PRIMARY_USER] = userStyle; sheet[CSS_PRIMARY_USER_IMPORTANT] = userImportantStyle; - buildUserAgentStyle (); + //buildUserAgentStyle (); buildUserStyle (); } Cheers, Johannes