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

Noto Nastaliq Urdu has many phantom (empty) glyphs #212

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

Noto Nastaliq Urdu has many phantom (empty) glyphs #212

GoogleCodeExporter opened this issue Jun 8, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

Copying from
http://lists.cairographics.org/archives/cairo/2014-November/025824.html

Something I noticed while debugging
this is hb-view appears to be emitting a lot of empty glyphs. In large
documents this will unnecessarily increase the file size and increase
rendering time.

Adding some debug to cairo_show_text_glyphs and using the sample text:

کے

I get:

cairo_show_text_glyphs   num_glyphs: 5
   glyph 0: 1005
   glyph 1: 1006
   glyph 2: 1007
   glyph 3: 248
   glyph 4: 409

Using ttx I can see that

 <mtx name="glyph01005" width="0" lsb="0"/>
 <mtx name="glyph01006" width="0" lsb="0"/>
 <mtx name="glyph01007" width="0" lsb="0"/>

and

<TTGlyph name="glyph01005" xMin="0" yMin="0" xMax="0" yMax="0">
  <contour>
    <pt x="0" y="0" on="1"/>
  </contour>
  <instructions><assembly>
  </assembly></instructions>
</TTGlyph>

<TTGlyph name="glyph01006" xMin="0" yMin="0" xMax="0" yMax="0">
  <contour>
    <pt x="0" y="0" on="1"/>
  </contour>
  <instructions><assembly>
    </assembly></instructions>
</TTGlyph>

<TTGlyph name="glyph01007" xMin="0" yMin="0" xMax="0" yMax="0">
  <contour>
    <pt x="0" y="0" on="1"/>
  </contour>
  <instructions><assembly>
  </assembly></instructions>
</TTGlyph>

Original issue reported on code.google.com by behdad@google.com on 25 Nov 2014 at 7:04

@behdad
Copy link
Contributor

behdad commented Jun 8, 2015

cc @behdad

@ghost ghost added the Script-Urdu label Jul 23, 2015
@ghost ghost added this to the Phase II - TTF milestone Jul 23, 2015
@kmansourMT
Copy link

The font includes a considerable number of what-seem-to-be-empty glyphs that function as control characters, and are required for correct rendering.

@behdad
Copy link
Contributor

behdad commented Apr 17, 2017

While it's possible to remove those, I don't think there's any problem with leaving them there.

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

4 participants