Re: Industry Thai Cell-Clustering Rules

[Theppitak Karoonboonayanan <thep links nectec or th>]

On Wed, Nov 08, 2000 at 11:10:49AM -0800, Chookij Vanatham wrote:
> 
> I agree with the modification.
> Let's test it and make sure it works.

Some questions regarding the changes :


---- 8< ----
@@ -363,7 +362,7 @@
 #ifdef	WTT_CLUSTERING
     "tis620-0",
     "tis620-1",
-    "tis620-2",
+    "tis620.2533-0",
 #endif
     "iso10646-1",
   };

Why tis620.2533-0 instead of tis620-2 here? Or is it just temporary?

---- 8< ----
@@ -451,7 +450,11 @@
       glyphs->glyphs[index].geometry.width = 
 	MAX (logical_rect.width, glyphs->glyphs[index - 1].geometry.width);
       glyphs->glyphs[index - 1].geometry.width = 0;
+#ifdef	WTT_CLUSTERING
+      glyphs->glyphs[index].geometry.x_offset = glyphs->glyphs[index].geometry.width;
+#else
       glyphs->glyphs[index].geometry.x_offset = 0;
+#endif

How about just this for combining case?

       glyphs->glyphs[index].geometry.x_offset = 0;
       glyphs->glyphs[index].geometry.width = 0;

---- 8< ----
@@ -806,8 +809,8 @@
 gboolean
 IsWttComposible(gint cur_wc, gint nxt_wc)
 {
-  switch (_TAC_celltype_inputcheck[ChrType(ucs2tis(cur_wc))]
-				  [ChrType(ucs2tis(nxt_wc))])
+  switch (_TAC_celltype_inputcheck[ChrType(cur_wc - 0xE00 + 0xA0)]
+				  [ChrType(nxt_wc - 0xE00 + 0xA0)])
   {
      case 'A':
      case 'S':

Why not using the ucs2tis() macro? Or is there any flaw I can't see?

-Theppitak.