[Webkit-unassigned] [Bug 28155] Remove platform-specific code from FontFastPath.cpp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 10 10:50:49 PDT 2009


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





--- Comment #1 from Yong Li <yong.li at torchmobile.com>  2009-08-10 10:50:48 PDT ---
(In reply to comment #0)
> http://trac.webkit.org/projects/webkit/changeset/46938 introduced
> WinCE-specific code to Font::glyphDataForCharacter(). The code seems misplaced,
> given that no other platform-specific code exists in FontFastPath.cpp. Whatever
> problem it is trying to solve should probably be addressed by platform-specific
> GlyphPageTreeNode code.

Do you also think a lot of #if PLATFORM(...) should be removed from those
platform-independent files? 

I can find tons of those.

for example, #if PLATFORM(MAC) in AccessibilityRenderObject.cpp

-- 
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