jabranr / noto

Automatically exported from code.google.com/p/noto
0 stars 0 forks source link

Code review request #301

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Branch name: dev_dougfelt_lint r95fdc96c1830

Purpose of code changes on this branch: new feature for summary/compare_summary

A number of fonts had significant changes in size but no obvious cause in the 
summary output.  It turned out that we'd recently asked for glyph names in the 
font, which caused a large increase in the size of the 'post' table. It seemed 
like adding the table sizes to the summary info and printing the names of 
tables with the largest size change would help clarify the source of the font 
size change.

When reviewing my code changes, please focus on: the usual

After the review, I'll merge this branch into:
n/a

Original issue reported on code.google.com by dougf...@google.com on 17 Mar 2015 at 12:39

GoogleCodeExporter commented 9 years ago
For comparing fonts when one has glyph names and another doesn't, it may be a 
good idea to convert the name table to format 3.0 and then compare then. The 
subsetter can do that.

Original comment by roozbeh@google.com on 17 Mar 2015 at 2:50

GoogleCodeExporter commented 9 years ago
Created a new issue to follow up on suggestion for comparing glyph names.

Original comment by dougf...@google.com on 3 Apr 2015 at 12:58