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

[bugzilla-daemon at webkit.org]

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





--- Comment #6 from Yong Li <yong.li at torchmobile.com>  2009-08-10 11:14:12 PDT ---
(In reply to comment #4)
> (In reply to comment #2)
> > I understand we should try to avoid using PLATFORM(). but apparently it's not
> > banned in webkit code.
> 
> I think a good rule to follow is:
> “If you think you need to add platform-specific code to a file that currently
> has no platform-specific code, then think again”.

If I could find a way to implement the functionality without changing
FontFastPath.cpp, I would definitely leave FontFastPath.cpp alone.

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