commit 5f85c80a07dd2d18348824866bf4e984ac711a24
author Behdad Esfahbod <behdad@behdad.org> Wed Jun 26 20:14:18 2013 -0400
committer Behdad Esfahbod <behdad@behdad.org> Wed Jun 26 20:14:18 2013 -0400
tree 0e752986560f9b8c8981232cc65f5d28d5328d19
parent 3f9e2dced298c3d00f31b2dfc38685bb071a3a22

[OT] Collect requiredFeature only if features are not provided

As per Werner's report on the list.

src/hb-ot-layout.cc

diff --git a/src/hb-ot-layout.cc b/src/hb-ot-layout.cc
index 37903e5..520deff 100644
--- a/src/hb-ot-layout.cc
+++ b/src/hb-ot-layout.cc
@@ -446,19 +446,19 @@
 					const hb_tag_t *features,
 					hb_set_t       *lookup_indexes /* OUT */)
 {
-  unsigned int required_feature_index;
-  if (hb_ot_layout_language_get_required_feature_index (face,
-							table_tag,
-							script_index,
-							language_index,
-							&required_feature_index))
-    _hb_ot_layout_collect_lookups_lookups (face,
-					   table_tag,
-					   required_feature_index,
-					   lookup_indexes);
-
   if (!features)
   {
+    unsigned int required_feature_index;
+    if (hb_ot_layout_language_get_required_feature_index (face,
+							  table_tag,
+							  script_index,
+							  language_index,
+							  &required_feature_index))
+      _hb_ot_layout_collect_lookups_lookups (face,
+					     table_tag,
+					     required_feature_index,
+					     lookup_indexes);
+
     /* All features */
     unsigned int feature_indices[32];
     unsigned int offset, len;