[Webkit-unassigned] [Bug 108881] Canvas fillText and measureText handle ideographic spaces differently

[bugzilla-daemon at webkit.org]

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





--- Comment #43 from Christophe Dumez <dchris at gmail.com>  2013-08-16 08:20:13 PST ---
(In reply to comment #42)
> In the case of find and memcpy, we are parsing a part of the string twice.  In the case of using for loop to find the first occurrence of space, we are parsing the string only once. So, for loop should be better to find the first occurrence of space?

I suggested to use String::charactersWithNullTermination() to initialize the Vector to avoid the memcpy already.

Regarding the find(), it returns as soon as it finds a replaceable space so it behaves exactly the same as your first loop. Duplicating code is not a good idea.

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