[HarfBuzz] Typo in harfbuzz-hangul.c

[Dohyun Kim]

Hi,

Recently I am interested in rendering of old Hangul text, which is
composed of conjoining Jamo characters (U+1100 and some other unicode
blocks).  For proper old Hangul rendering, several opentype features
must be supported by truetype or opentye fonts.  In addition,
rendering engine should also support these features.  The ICU layout
engine can already do this, which however is not the case of Pango
yet.

Anyway, today I have download Harfbuzz source tree by git and found an
obvious typo in harfbuzz-hangul.c, though I do not know anything about
C language.

--- harfbuzz.orig/src/harfbuzz-hangul.c	2010-04-17 10:37:08.347863347 +0900
+++ harfbuzz/src/harfbuzz-hangul.c	2010-04-17 10:53:18.978819261 +0900
@@ -130,7 +130,7 @@
 static const HB_OpenTypeFeature hangul_features [] = {
     { HB_MAKE_TAG('c', 'c', 'm', 'p'), CcmpProperty },
     { HB_MAKE_TAG('l', 'j', 'm', 'o'), CcmpProperty },
-    { HB_MAKE_TAG('j', 'j', 'm', 'o'), CcmpProperty },
+    { HB_MAKE_TAG('v', 'j', 'm', 'o'), CcmpProperty },
     { HB_MAKE_TAG('t', 'j', 'm', 'o'), CcmpProperty },
     { 0, 0 }
 };

As shown in http://www.microsoft.com/Typography/OTSpec/featurelist.htm,
"vjmo", instead of "jjmo", is a required feature for rendering old
Hangul characters.  It seems that this affects the Google Chrome (I am
using ver 5.0.342.9 beta on Fedora Linux), which currently does not
render old Hangul correctly.

I hope that in the near future these features are supported by Pango as well.

Best Regards,
-- 
Dohyun Kim