Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

GSUB (and GPOS?) bloated #105

Closed
GoogleCodeExporter opened this issue Jun 8, 2015 · 3 comments
Closed

GSUB (and GPOS?) bloated #105

GoogleCodeExporter opened this issue Jun 8, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

As I reported here:
https://github.com/adobe-fonts/source-han-sans/issues/46



I see multiple copies of the same features. Eg:

  <FeatureRecord index="0"> 
    <FeatureTag value="aalt"/> 
    <Feature> 
      <!-- LookupCount=2 --> 
      <LookupListIndex index="0" value="0"/> 
      <LookupListIndex index="1" value="1"/> 
    </Feature> 
  </FeatureRecord> 
  <FeatureRecord index="1"> 
    <FeatureTag value="aalt"/> 
    <Feature> 
      <!-- LookupCount=2 --> 
      <LookupListIndex index="0" value="0"/> 
      <LookupListIndex index="1" value="1"/> 
    </Feature> 
  </FeatureRecord> 
  <FeatureRecord index="2"> 
    <FeatureTag value="aalt"/> 
    <Feature> 
      <!-- LookupCount=2 --> 
      <LookupListIndex index="0" value="0"/> 
      <LookupListIndex index="1" value="1"/> 
    </Feature> 
  </FeatureRecord> 
  <FeatureRecord index="3"> 
    <FeatureTag value="aalt"/> 
    <Feature> 
      <!-- LookupCount=2 --> 
      <LookupListIndex index="0" value="0"/> 
      <LookupListIndex index="1" value="1"/> 
    </Feature> 
  </FeatureRecord> 
  <FeatureRecord index="4"> 
    <FeatureTag value="aalt"/> 
    <Feature> 
      <!-- LookupCount=2 --> 
      <LookupListIndex index="0" value="0"/> 
      <LookupListIndex index="1" value="1"/> 
    </Feature> 
  </FeatureRecord> 

It makes reading the GSUB tables very hard unnecessarily.
The snippet above is from NotoSansCJK-Regular, but I suppose it's the same with 
Source Han Sans.

Original issue reported on code.google.com by behdad@google.com on 31 Jul 2014 at 8:27

@GoogleCodeExporter
Copy link
Author

Original comment by stua...@google.com on 28 Feb 2015 at 12:06

  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@behdad
Copy link
Contributor

behdad commented Jun 8, 2015

cc @behdad @kenlunde

@ghost
Copy link

ghost commented Jun 9, 2015

Moved to notofonts/noto-cjk#9

@ghost ghost closed this as completed Jun 9, 2015
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants