Bug 123928 - Cleanup FontMac.mm
Summary: Cleanup FontMac.mm
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Daniel Bates
Depends on:
Reported: 2013-11-06 14:20 PST by Daniel Bates
Modified: 2013-11-06 15:58 PST (History)
1 user (show)

See Also:

Patch (3.86 KB, patch)
2013-11-06 14:22 PST, Daniel Bates
aestes: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2013-11-06 14:20:54 PST
We should remove the default case statement from the switch block in Font::drawGlyphs(), <http://trac.webkit.org/browser/trunk/Source/WebCore/platform/graphics/mac/FontMac.mm?rev=158581#L143>, so that the compiler enforces that the switch covers all cases. We should inline the definition of isVertical in showGlyphsWithAdvances() since its referenced exactly once and its value is sufficiently clear. Additionally, we should use the variable platformData throughout Font::drawGlyphs() instead of using the variable in some place and explicitly querying for the Font's platform data in others.
Comment 1 Daniel Bates 2013-11-06 14:22:00 PST
Created attachment 216226 [details]
Comment 2 Daniel Bates 2013-11-06 15:58:22 PST
Committed r158800: <http://trac.webkit.org/changeset/158800>