[Webkit-unassigned] [Bug 79216] [BlackBerry] Upstream helper classes for skia
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Feb 23 19:24:32 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=79216
--- Comment #2 from Antonio Gomes <tonikitoo at webkit.org> 2012-02-23 19:24:32 PST ---
(From update of attachment 128166)
View in context: https://bugs.webkit.org/attachment.cgi?id=128166&action=review
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:100
> + FcBool b;
> + int i;
b and i does not make it easy to understand their use.
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:114
> +#if 0
> + // FIXME: Fontconfig tells us to use aggressive hinting, but FreeType breaks with our Adobe fonts
> + if (FcPatternGetInteger(match, FC_HINT_STYLE, 0, &i) == FcResultMatch)
> + out->hintStyle = i;
> +#endif
up to drop this comment?
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:147
> + if (U16_IS_SURROGATE(characters[i])
> + && U16_IS_SURROGATE_LEAD(characters[i])
> + && i != numCharacters - 1
> + && U16_IS_TRAIL(characters[i + 1])) {
> + FcCharSetAddChar(cset, U16_GET_SUPPLEMENTARY(characters[i], characters[i+1]));
> + i++;
> + } else
> + FcCharSetAddChar(cset, characters[i]);
wrong code style (indentation)
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:198
> + }
> + int weight;
add blank line between these two
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:203
> + family->isBold = false;
> + int slant;
ditto
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.cpp:208
> + family->isItalic = false;
> + FcFontSetDestroy(fontSet);
ditto
> Source/WebCore/platform/graphics/blackberry/skia/PlatformSupport.h:43
> +// This is a minimal version of the Chromium PlatformSupport/WebFontInfo classes used for font support
> +
> +class PlatformSupport {
drop blank line, add "." at the end
--
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