[Webkit-unassigned] [Bug 48578] HTML5 Conformance Test failure: approved/canvas/canvas_text_font_001.htm
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jun 7 11:11:08 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=48578
--- Comment #21 from Mustafizur Rahaman <mustaf.here at gmail.com> 2011-06-07 11:11:08 PST ---
(In reply to comment #20)
> (From update of attachment 96217 [details])
> I don't understand why we need to remove/append to substrings for your algorithm to work.
i needed to use append/remove to get the exact substring(I was getting my substring separated by space) to be compared with "inherit"/"initial".
>
> Isn't it just:
>
> while (start < end) {
> if (isSpaceOrNewLine(*start))
> continue;
> bool equalIgnoringCase(start, "inherit") || equalIgnoringCase(start, "initial") {
> start += 6;
> if (start == end || isSpaceOrNewline(start))
> return true;
> }
> }
>
Though I am not completely sure which equalIgnoringCase() you have suggested me to use, but from my understanding I came up with the below algo.
What is your thoughts on this?
while (fontStringStart <= fontStringEnd) {
if (isSpaceOrNewline((*stringImpl)[fontStringStart])) {
fontStringStart++;
continue;
}
if (equalIgnoringCase("inherit",stringImpl->characters() + fontStringStart,7)
|| equalIgnoringCase("initial",stringImpl->characters() + fontStringStart, 7)) {
fontStringStart = fontStringStart + 7;
if (fontStringStart > fontStringEnd || isSpaceOrNewline((*stringImpl)[fontStringStart]))
return true;
}
fontStringStart++;
}
return false;
--
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