Hi Bram, Bram Moolenaar <[email protected]> wrote: > Ali Gholami Rudi wrote: > > Farsi is quite similar to arabic except for a few letters. This patch > > merges farsi support into arabic.c. AFAICS, farsi.c and fkmap option > > seem to be obsolete (unless someone is still using the pcf fonts). > > > > Also it might make sense to add a new farsi option, just like the arabic > > option, that when set implies "set rl" and "set keymap=persian". > > I don't use these languages, thus I need to rely on others to tell me > what works best.
I've CC'ed the authors of arabic.c and farsi.c. > After including this patch, can we drop farsi.c? Part of it implements the input method which is now done separately using the keymap option; those parts *are* obsolete. Also it uses Farsi ISIR-3342 encoding (is there anyone who uses that encoding?) while arabic.c uses utf-8. If the included farsi fonts are for that specific encoding, it might make sense to drop farsi.c, fkmap option and the fonts altogether (but I might be missing something). Mortaza, what do you think? Anyway, AFAICT currently the only option for those how use utf-8 is "set rl". But it fails to combine farsi-specific characters and this patch fixes it. Note that farsi input method is already available using "set keymap=persian". > There should also be an update to the documentation, most users don't > look at the source code to figure out what to do :-). Does it make sense to add some sections to arabic.txt? Most of it applies to farsi, too. Regards, Ali --~--~---------~--~----~------------~-------~--~----~ You received this message from the "vim_dev" maillist. For more information, visit http://www.vim.org/maillist.php -~----------~----~----~----~------~----~------~--~---
