[Webkit-unassigned] [Bug 70107] Custom written CSS lexer

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 11 04:50:41 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=70107





--- Comment #46 from Zoltan Herczeg <zherczeg at webkit.org>  2012-01-11 04:50:39 PST ---
> > Source/WebCore/css/CSSParser.cpp:263
> > -    int length = string.length() + strlen(prefix) + strlen(suffix) + 2;
> > +    int length = string.length() + strlen(prefix) + strlen(suffix) + 1;
> 
> Why have we dropped to only a single extra null terminator?  I recall (though haven't got that far in the patch) that we rely on two nulls at the end of the string

The double zero terminator is only required by Flex. The attached code does not depend on the double terminator (thanks to C short circuit evaluation model).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list