From eafd163bc5c3568763b0b0ebb3d39041e1290b3b Mon Sep 17 00:00:00 2001
From: handa <handa>
Date: Mon, 4 Oct 2004 00:37:44 +0000
Subject: [PATCH] (UPDATE_CLUSTER_RANGE): New macro. (run_rule): Don't update
 cluster range here. (run_command): Update cluster range on
 appending a glyph. (run_otf): Update cluster range for
 glyphs generated by OTF.

---
 src/font-flt.c |   25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/src/font-flt.c b/src/font-flt.c
index c8172ff..c6a7e5b 100644
--- a/src/font-flt.c
+++ b/src/font-flt.c
@@ -263,6 +263,17 @@ static MSymbol Mcond, Mrange;
 
 #define GLYPH_CODE_INDEX(code) ((code) - GLYPH_CODE_MIN)
 
+#define UPDATE_CLUSTER_RANGE(ctx, g)		\
+  do {						\
+    if ((ctx)->cluster_begin_idx)		\
+      {						\
+	if (ctx->cluster_begin_pos > (g).pos)	\
+	  ctx->cluster_begin_pos = (g).pos;	\
+	if (ctx->cluster_end_pos < (g).to)	\
+	  ctx->cluster_end_pos = (g).to;	\
+      }						\
+  } while (0);
+
 enum FontLayoutCmdRuleSrcType
   {
     SRC_REGEX,
@@ -1078,13 +1089,6 @@ run_rule (int depth,
 	    continue;
 	  i--;
 	}
-      if (ctx->cluster_begin_idx)
-	{
-	  if (ctx->cluster_begin_pos > MGLYPH (from)->pos)
-	    ctx->cluster_begin_pos = MGLYPH (from)->pos;
-	  if (ctx->cluster_end_pos < MGLYPH (to)->pos)
-	    ctx->cluster_end_pos = MGLYPH (to)->pos;
-	}
       pos = run_command (depth, rule->cmd_ids[i], gstring, from, to, ctx);
       if (pos < 0)
 	MERROR (MERROR_DRAW, -1);
@@ -1127,6 +1131,8 @@ run_otf (int depth,
 	 FontLayoutContext *ctx)
 {
 #ifdef HAVE_OTF
+  int from_idx = gstring->used;
+
   MDEBUG_PRINT4 ("\n [FLT] %*s(OTF %s,%s)", depth, "",
 		 (otf_cmd->gsub_features == Mnil ? ""
 		  : MSYMBOL_NAME (otf_cmd->gsub_features)),
@@ -1135,6 +1141,9 @@ run_otf (int depth,
   to = mfont__ft_drive_otf (gstring, from, to,
 			    otf_cmd->script, otf_cmd->langsys,
 			    otf_cmd->gsub_features, otf_cmd->gpos_features);
+  if (ctx->cluster_begin_idx)
+    for (; from_idx < gstring->used; from_idx++)
+      UPDATE_CLUSTER_RANGE (ctx, gstring->glyphs[from_idx]);
 #endif
   return to;
 }
@@ -1172,6 +1181,7 @@ run_command (int depth, int id, MGlyphString *gstring, int from, int to,
 	    g.to = tmp->to;
 	}
       APPEND_GLYPH (gstring, g);
+      UPDATE_CLUSTER_RANGE (ctx, g);
       ctx->code_offset = ctx->combining_code = ctx->left_padding = 0;
       MDEBUG_PRINT (")");
       return (from);
@@ -1215,6 +1225,7 @@ run_command (int depth, int id, MGlyphString *gstring, int from, int to,
 	if (ctx->left_padding)
 	  g.left_padding = ctx->left_padding;
 	APPEND_GLYPH (gstring, g);
+	UPDATE_CLUSTER_RANGE (ctx, g);
 	MDEBUG_PRINT3 ("\n [FLT] %*s(COPY 0x%X)", depth, "", g.code);
 	ctx->code_offset = ctx->combining_code = ctx->left_padding = 0;
 	return (from + 1);
-- 
1.7.10.4