commit eb0bf3ae6688b7e98a706df2ad2714c071d77e22
author Behdad Esfahbod <behdad@behdad.org> Wed Aug 06 15:36:41 2014 -0400
committer Behdad Esfahbod <behdad@behdad.org> Thu Apr 09 19:16:57 2015 -0700
tree 81d0d6e190cf205fd7566236c5925056f942bd2a
parent e8fd83932a75cfbaa4638a757868915ebfac3c1f

Relax inert checks

Previously, when creating an object from inert inputs (eg:
"hb_font_create(hb_face_get_empty())") we returned the inert
empty object.  This is not helpful as there are legitimate
usecases to do that.

We now never return the inert object unless allocation failed.

Tests are revised to reflect.

src/hb-font.cc

diff --git a/src/hb-font.cc b/src/hb-font.cc
index d42db59..3df41bd 100644
--- a/src/hb-font.cc
+++ b/src/hb-font.cc
@@ -854,8 +854,6 @@
 
   if (unlikely (!face))
     face = hb_face_get_empty ();
-  if (unlikely (hb_object_is_inert (face)))
-    return hb_font_get_empty ();
   if (!(font = hb_object_create<hb_font_t> ()))
     return hb_font_get_empty ();
 
@@ -880,7 +878,7 @@
 hb_font_create_sub_font (hb_font_t *parent)
 {
   if (unlikely (!parent))
-    return hb_font_get_empty ();
+    parent = hb_font_get_empty ();
 
   hb_font_t *font = hb_font_create (parent->face);