From 51ce3828e67a8ca7e4ea4b69e88c395e243bf01f Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Fri, 19 Jan 2018 18:36:39 -0800
Subject: [PATCH] [aat] Fix Ligature matching

I hope... Makes "ffi" work with Zapfino. I'm not sure if doing it correctly though...
---
 src/hb-aat-layout-morx-table.hh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/hb-aat-layout-morx-table.hh b/src/hb-aat-layout-morx-table.hh
index aa7b6d90e..c64dca066 100644
--- a/src/hb-aat-layout-morx-table.hh
+++ b/src/hb-aat-layout-morx-table.hh
@@ -362,7 +362,7 @@ struct LigatureSubtable
 	  if (unlikely (!match_length))
 	    return false;
 
-	  buffer->move_to (match_positions[match_length - 1]);
+	  buffer->move_to (match_positions[--match_length]);
 
 	  const HBUINT32 &actionData = ligAction[action_idx];
 	  if (unlikely (!actionData.sanitize (&c->sanitizer))) return false;
@@ -384,6 +384,7 @@ struct LigatureSubtable
 	    if (unlikely (!ligatureData.sanitize (&c->sanitizer))) return false;
 	    hb_codepoint_t lig = ligatureData;
 
+	    match_positions[match_length++] = buffer->out_len;
 	    buffer->replace_glyph (lig);
 
 	    //ligature_idx = 0; // XXX Yes or no?
@@ -395,7 +396,6 @@ struct LigatureSubtable
 	  }
 	  /* TODO merge_clusters / unsafe_to_break */
 
-	  match_length--;
 	  action_idx++;
 	}
 	while (!(action & LigActionLast));