[Webkit-unassigned] [Bug 22339] Support Indian web sites with EOT by on-the-fly transcoding to Unicode

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 16 00:47:46 PST 2009


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





------- Comment #12 from prunthaban at google.com  2009-02-16 00:47 PDT -------
(In reply to comment #11)
> (From update of attachment 26892 [review])
> Wow, if the world ever needed a for loop, looks like this might be it!
> 
>    m_lookupTable.set(String(in + 0, 1), String(out + 0, 1));
>  125     m_lookupTable.set(String(in + 1, 1), String(out + 1, 1));
>  126 
> 
The problem is the Strings are not always of length 1 and they do not follow an
order. I should have still had the length in an array and used a for loop. But
then we will ultimately end up having two extra arrays with numbers filled in
and a loop accumulating the start index. This part of the code was generated
from Padma JS file using an automated program I have written (so that it will
be easier to do the same for other fonts) and it was easier to keep it this
way. If for readability, the loop based approach is preferred, I will change
it.


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



More information about the webkit-unassigned mailing list