Instead of calling $table->getFont() twice for each outline, pass it along to init and parse functions, reduces overall parsing time by another 20% for FreeSerif

Author: Indrek Altpere

src/FontLib/Glyph/Outline.php

@@ -42,8 +42,7 @@ class Outline extends BinaryStream {
    *
    * @return Outline
    */
-  static function init(glyf $table, $offset, $size) {
-    $font = $table->getFont();
+  static function init(glyf $table, $offset, $size, BinaryStream $font) {
     $font->seek($offset);
 
     if ($font->readInt16() > -1) {
@@ -55,7 +54,7 @@ static function init(glyf $table, $offset, $size) {
       $glyph = new OutlineComposite($table, $offset, $size);
     }
 
-    $glyph->parse();
+    $glyph->parse($font);
 
     return $glyph;
   }
@@ -73,8 +72,7 @@ function __construct(glyf $table, $offset = null, $size = null) {
     $this->size   = $size;
   }
 
-  function parse() {
-    $font = $this->getFont();
+  function parse(BinaryStream $font) {
     $font->seek($this->offset);
 
     if (!$this->size) {

src/FontLib/Table/Type/glyf.php

@@ -31,7 +31,7 @@ protected function _parse() {
     foreach ($real_loca as $gid => $location) {
       $_offset    = $offset + $loca[$gid];
       $_size      = $loca[$gid + 1] - $loca[$gid];
-      $data[$gid] = Outline::init($this, $_offset, $_size);
+      $data[$gid] = Outline::init($this, $_offset, $_size, $font);
     }
 
     $this->data = $data;