Merge pull request #305 from robotools/justvanrossum-patch-1

Pass flags down when copying the data

Author: justvanrossum

Lib/fontMath/mathGlyph.py

@@ -131,7 +131,7 @@ def __ne__(self, other):
 
     def copy(self):
         """return a new MathGlyph containing all data in self"""
-        return MathGlyph(self)
+        return MathGlyph(self, scaleComponentTransform=self.scaleComponentTransform, strict=self.strict)
 
     def copyWithoutMathSubObjects(self):
         """
@@ -143,7 +143,7 @@ def copyWithoutMathSubObjects(self):
 
         this is used mainly for internal glyph math.
         """
-        n = MathGlyph(None)
+        n = MathGlyph(None, scaleComponentTransform=self.scaleComponentTransform, strict=self.strict)
         n.name = self.name
         if self.unicodes is not None:
             n.unicodes = list(self.unicodes)

Lib/fontMath/test/test_mathGlyph.py

@@ -509,6 +509,12 @@ def test_image_round(self):
         glyph2 = glyph1.round()
         self.assertEqual(glyph2.image, expected)
 
+    def test_copy(self):
+        glyph1 = self._setupTestGlyph()
+        glyph1.unicodes = []
+        glyph2 = glyph1.copy()
+        self.assertEqual(glyph1, glyph2)
+
 
 class MathGlyphPenTest(unittest.TestCase):
     def __init__(self, methodName):