From 5132bbe411c8dea05c3a12d3b64905a3d23eccd5 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Mon, 20 Feb 2017 12:36:15 -0600
Subject: [PATCH] [symfont] Remove glyph perimeter from GlyphStatistics

It's not technically statistics the same way the other properties are, and
we have a separate pen for it, so remove. Goal is to turn GlyphStatistics
into a pen.

Also remove perimeter from interpolatable.py contour characteristics vector.
Shouldn't be needed, given all the stats we already include.
---
 Snippets/interpolatable.py | 1 -
 Snippets/symfont.py        | 7 -------
 2 files changed, 8 deletions(-)

diff --git a/Snippets/interpolatable.py b/Snippets/interpolatable.py
index 6792f06e3..10eeab012 100755
--- a/Snippets/interpolatable.py
+++ b/Snippets/interpolatable.py
@@ -137,7 +137,6 @@ def test(glyphsets, glyphs=None, names=None):
 					stats = GlyphStatistics(contour, glyphset=glyphset)
 					size = abs(stats.Area) ** .5 * .5
 					vector = (
-						int(stats.Perimeter * .125),
 						int(size),
 						int(stats.MeanX),
 						int(stats.MeanY),
diff --git a/Snippets/symfont.py b/Snippets/symfont.py
index 91cfe5fc7..298540aa9 100755
--- a/Snippets/symfont.py
+++ b/Snippets/symfont.py
@@ -15,7 +15,6 @@ import sys
 import math
 from fontTools.pens.basePen import BasePen
 from fontTools.pens.transformPen import TransformPen
-from fontTools.pens.perimeterPen import PerimeterPen
 from fontTools.pens.momentsPen import MomentsPen
 from fontTools.pens.areaPen import AreaPen
 from fontTools.misc.transform import Scale
@@ -201,12 +200,6 @@ class GlyphStatistics(object):
 		self._glyphset = glyphset
 		self._transform = transform
 
-		pen = transformer = PerimeterPen(glyphset=self._glyphset)
-		if self._transform:
-			transformer = TransformPen(pen, self._transform)
-		self._glyph.draw(transformer)
-		self.Perimeter = pen.value
-
 		Pen = MomentsPen
 		pen = transformer = Pen(glyphset=self._glyphset)
 		if self._transform: