all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Agustin Martin <agustin.martin@hispalinux.es>
To: 11200-done@debbugs.gnu.org
Subject: bug#11200: 24.0.95; ispell.el sets incorrect encoding for the default dictionary
Date: Mon, 16 Apr 2012 19:05:01 +0200	[thread overview]
Message-ID: <20120416170501.GA14907@agmartin.aq.upm.es> (raw)
In-Reply-To: <83aa2ma7sk.fsf@gnu.org>

On Sun, Apr 08, 2012 at 12:31:55PM +0300, Eli Zaretskii wrote:
> In ispell.el we see:
> 
>   (defvar ispell-dictionary-base-alist
>     '((nil
>        ;; The default dictionary.  It may be English.aff, or any other
>        ;; dictionary depending on locale and such things.  We should probably
>        ;; ask ispell what dictionary it's using, but until we do that, let's
>        ;; just use an approximate regexp.
>        "[[:alpha:]]" "[^[:alpha:]]" "[']" nil ("-B") nil iso-8859-1)
> 
> The iso-8859-1 part is not the optimal choice for aspell and hunspell.
> These spell-checkers support UTF-8 encoding, so using utf-8 here will
> DTRT in a much larger number of locales.  With iso-8859-1, any
> non-Latin locale will require the user to customize
> ispell-local-dictionary-alist, when there's no real reason for forcing
> them do that.
> 
> Please include some simple logic to use utf-8 with the default
> dictionary, when the speller is aspell or hunspell.

Committed change to use utf-8 for recent aspell or hunspell together with
the change to use [:alpha:] if available. Closing bug report.

By the way, since global dicts will always use [:alpha:] in Emacs, I will
revert above `ispell-dictionary-base-alist' definition to the XEmacs
friendly original version. Should have no effect at all for Emacs.

Thanks for your hunspell tests and general feedback.

-- 
Agustin





      reply	other threads:[~2012-04-16 17:05 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-08  9:31 bug#11200: 24.0.95; ispell.el sets incorrect encoding for the default dictionary Eli Zaretskii
2012-04-16 17:05 ` Agustin Martin [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120416170501.GA14907@agmartin.aq.upm.es \
    --to=agustin.martin@hispalinux.es \
    --cc=11200-done@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.