Re: [ft-devel] autofit CJK module

[Werner LEMBERG]

> I've just committed autofit CJK module.  It is based on akito's
> autohint patch.

Looks great!  Thanks for your work.

> The CJK module is currently in experimental stage and is disabled by
> default.  If you are interested in helping or want to have a try on
> it, it can enabled by #define AF_MOD_CJK in src/autofit/afcjk.c.

Please move AF_MOD_CJK to ftoption.h and enable it by default.  This
makes it easier to test.

BTW, I see code like this in the new module:

  FT_Pos  threshold = 50 * metrics->units_per_em / 2048;

Where does the `2048' come from?  Please add a comment.


    Werner