code review 14494062: freetype/truetype: propagate the useMyMetrics flag to n...

freetype/truetype: propagate the useMyMetrics flag to nested components.

Also increase the maximum recursion depth from 4 to 8, since some
x-deja-vu-sans-oblique glyphs are actually depth 4. It's an arbitrary
limit. The C Freetype code doesn't seem to limit recursion (below
whatever the font says it needs), but it seems safer to have a limit.

[nigeltao, 2013-10-16 10:46:23]

Hello bsiegert@gmail.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/freetype-go

[bsiegert, 2013-10-16 11:12:19]

LGTM

You may want to set the max recursion depth even higher so that J. Random User
does not run into this with his fonts.

[nigeltao, 2013-10-16 23:21:36]

*** Submitted as
https://code.google.com/p/freetype-go/source/detail?r=26847f1046e3 ***

freetype/truetype: propagate the useMyMetrics flag to nested components.

Also increase the maximum recursion depth from 4 to 8, since some
x-deja-vu-sans-oblique glyphs are actually depth 4. It's an arbitrary
limit. The C Freetype code doesn't seem to limit recursion (below
whatever the font says it needs), but it seems safer to have a limit.

R=bsiegert
CC=golang-dev
https://codereview.appspot.com/14494062