Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1971)

Issue 301820043: Issue 4868: Add limited embedding support for OpenType/CFF Collection (OTC) fonts

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 4 months ago by trueroad
Modified:
3 years, 4 months ago
Reviewers:
lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Issue 4868/2: Add limited embedding support for OTC fonts This commit adds limited embedding support for OpenType/CFF Collection (OTC) fonts which have `*.otc' filename extension. Essentially, the extension of OTC font files should be `*.otc'. However, most of the OTC fonts have the extension `*.ttc' for backward compatibility. https://www.microsoft.com/typography/otspec/recom.htm So this is limited support because most of the OTC fonts cannot be used. TODO: distinguish files which have extension `*.ttc' whether TrueType Collection (TTC) fonts or OTC fonts. Issue 4868/1: Add font-index for embedding OTC fonts This commit adds support font-index for OpenType/CFF Collection (OTC) fonts not only TrueType Collection (TTC) fonts.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -4 lines) Patch
M lily/pfb-scheme.cc View 2 chunks +34 lines, -3 lines 0 comments Download
M scm/framework-ps.scm View 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 2
lemzwerg
LGTM, thanks!
3 years, 4 months ago (2016-05-30 18:13:05 UTC) #1
trueroad
3 years, 4 months ago (2016-06-06 13:58:24 UTC) #2
I've pushed to staging.

Issue 4868/1: Add font-index for embedding OTC fonts
commit 8d1f238d09ada2152baebdfc949b7f56c2e7bfd6

Issue 4868/2: Add limited embedding support for OTC fonts
commit 44802bfc4e5c3c03e675a83a5afe7ac99fb3b26d
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b