# HG changeset patch
# User Kenichi Handa <handa@m17n.org>
# Date 1219996391 0
# Node ID a4677d55715f6676079ea931f94a63fbb51c233c
# Parent  f15876e666813ac9f4eebec4802a312a2f2a703d
(optimize_sub_char_table): Perform more greedy optimization.

diff -r f15876e66681 -r a4677d55715f src/chartab.c
--- a/src/chartab.c	Fri Aug 29 07:52:57 2008 +0000
+++ b/src/chartab.c	Fri Aug 29 07:53:11 2008 +0000
@@ -660,28 +660,27 @@
   struct Lisp_Sub_Char_Table *tbl = XSUB_CHAR_TABLE (table);
   int depth = XINT (tbl->depth);
   Lisp_Object elt, this;
-  int i;
+  int i, optimizable;
 
   elt = XSUB_CHAR_TABLE (table)->contents[0];
   if (SUB_CHAR_TABLE_P (elt))
     elt = XSUB_CHAR_TABLE (table)->contents[0]
       = optimize_sub_char_table (elt, test);
-  if (SUB_CHAR_TABLE_P (elt))
-    return table;
+  optimizable = SUB_CHAR_TABLE_P (elt) ? 0 : 1;
   for (i = 1; i < chartab_size[depth]; i++)
     {
       this = XSUB_CHAR_TABLE (table)->contents[i];
       if (SUB_CHAR_TABLE_P (this))
 	this = XSUB_CHAR_TABLE (table)->contents[i]
 	  = optimize_sub_char_table (this, test);
-      if (SUB_CHAR_TABLE_P (this)
-	  || (NILP (test) ? NILP (Fequal (this, elt)) /* defaults to `equal'. */
+      if (optimizable
+	  && (NILP (test) ? NILP (Fequal (this, elt)) /* defaults to `equal'. */
 	      : EQ (test, Qeq) ? !EQ (this, elt)      /* Optimize `eq' case.  */
 	      : NILP (call2 (test, this, elt))))
-	break;
+	optimizable = 0;
     }
 
-  return (i < chartab_size[depth] ? table : elt);
+  return (optimizable ? elt : table);
 }
 
 DEFUN ("optimize-char-table", Foptimize_char_table, Soptimize_char_table,