From 738866492ddfa98d75c9f5f399c662d9f958dd9b Mon Sep 17 00:00:00 2001
From: Cosimo Lupo <cosimo.lupo@daltonmaag.com>
Date: Tue, 8 Dec 2015 09:35:00 +0000
Subject: [PATCH] _g_l_y_f: expand glyphs inside `compile` if
 recalcBBoxes==True; delete empty self.data lingering around

Fixes https://github.com/behdad/fonttools/issues/410.

See also:
https://github.com/behdad/fonttools/commit/ecbe8f2c37021605672af3c9d4f6b0f6e73df592#commitcomment-14464893
---
 Lib/fontTools/ttLib/tables/_g_l_y_f.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Lib/fontTools/ttLib/tables/_g_l_y_f.py b/Lib/fontTools/ttLib/tables/_g_l_y_f.py
index 4d8bd1afb..6736dcf08 100644
--- a/Lib/fontTools/ttLib/tables/_g_l_y_f.py
+++ b/Lib/fontTools/ttLib/tables/_g_l_y_f.py
@@ -313,6 +313,7 @@ class Glyph(object):
 			return
 		if not self.data:
 			# empty char
+			del self.data
 			self.numberOfContours = 0
 			return
 		dummy, data = sstruct.unpack2(glyphHeaderFormat, self.data, self)
@@ -329,7 +330,11 @@ class Glyph(object):
 
 	def compile(self, glyfTable, recalcBBoxes=True):
 		if hasattr(self, "data"):
-			return self.data
+			if recalcBBoxes:
+				# must unpack glyph in order to recalculate bounding box
+				self.expand(glyfTable)
+			else:
+				return self.data
 		if self.numberOfContours == 0:
 			return ""
 		if recalcBBoxes: