[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#17742: Acknowledgement (Support for enchant?)

From: Reuben Thomas
Subject: bug#17742: Acknowledgement (Support for enchant?)
Date: Tue, 13 Dec 2016 00:53:47 +0000

On 6 December 2016 at 15:56, Reuben Thomas <address@hidden> wrote:
On 6 December 2016 at 15:55, Eli Zaretskii <address@hidden> wrote:

I wonder if it won't be better to delay pushing this until you solve
that problem.  The ispell-dictionary-alist is for Ispell speller only;
all the modern spellers should glean that information from the
installed dictionaries, and not bother users with tedious setups.  Do
you see any need to get this into master before you add this
functionality for enchant?

​Sorry if I wasn't clear: I'm working on adding dictionary detection right now.​ You're quite right that should be added before the patch is considered installable.
Thanks.  Note that this should not only list the installed
dictionaries, but also deduce the suitable values of CASECHARS and
NOT-CASECHARS for each dictionary.

​Thanks for the tip.

​In fact, it proves to be impossible to deduce suitable values currently, so I set defaults. Enchant will need changes to support this (unless one finds the provider for each dictionary and uses ispell.el's existing back-end-specific code, but this is a nasty solution). This simple solution gets the correct encoding at least (Enchant is always utf8), and hence fixes the misalignment errors.

However, I have updated my enchant patches to add dictionary listing, and hence my ispell.el patches to automatically find the enchant dictionaries.


Attachment: 0025-Add-Enchant-support-to-ispell.el.patch
Description: Text Data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]