On 01/06/09 12:41, Ali Gholami Rudi wrote:
> Hi,
>
> This patch replaces switch statements in arabic.c with a static array of
> struct achars.  This makes adding new letters a lot easier and that
> usually involves only adding an entry to this array (I've already added
> Farsi letters).
>
> This is also a clean up patch and reduces more than 1k lines.  After
> applying this patch arabic.h holds only a macro definition.  So it is
> probably a good idea to remove arabic.h completely and move the trivial
> ARABIC_CHAR macro to another header.
>
>       Ali
> ---
>   src/arabic.c | 1127 
> ++++++++++------------------------------------------------
>   src/arabic.h |  243 +-------------
>   src/mbyte.c  |   30 --
>   3 files changed, 189 insertions(+), 1211 deletions(-)
[...]

I haven't verified everything, but shouldn't it be
> +     {0x0624, 0xfe85, 0, 0, 0xfe85},                 /* a_WAW_HAMZA */
  +     {0x0624, 0xfe85, 0, 0, 0xfe86},         /* a_WAW_HAMZA */

(i.e., waw-hamza final U+FE86 different from isolated U+FE85) ?


Best regards,
Tony.
-- 
hundred-and-one symptoms of being an internet addict:
43. You tell the kids they can't use the computer because "Daddy's got 
work to
     do" and you don't even have a job.

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Raspunde prin e-mail lui