all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Agustin Martin <agustin.martin@hispalinux.es>
Cc: 6539@debbugs.gnu.org
Subject: bug#6539: ispell-complete-word looks for wrong file (at least	on	w32)
Date: Mon, 05 Jul 2010 20:23:49 +0300	[thread overview]
Message-ID: <83mxu59716.fsf@gnu.org> (raw)
In-Reply-To: <20100705115222.GA2013@agmartin.aq.upm.es>

> Date: Mon, 5 Jul 2010 13:52:22 +0200
> From: Agustin Martin <agustin.martin@hispalinux.es>
> Cc: 
> 
> However, docstrings and error handling for this complete-word stuff needs 
> to be improved, so it provides useful and accurate information. I am 
> currently trying something like attached patch where 
> `ispell-alternate-dictionary' is set to nil if no default is found and 
> checked before use. I find error message too long, but better that before.
> As usually, suggestions are welcome.

For the error message, just put 2 spaces between the sentences, as we
do everywhere in Emacs.

For the rest, the patch looks fine, but why not test /usr/dict/words
with file-readable-p as well?  Then, if it does not exist, the
defcustom will evaluate to nil, and the error message will be even
more to the point.

Thanks.





  parent reply	other threads:[~2010-07-05 17:23 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-30  9:18 bug#6539: ispell-complete-word looks for wrong file (at least on w32) Lennart Borgman
2010-06-30 10:38 ` Agustin Martin
2010-06-30 11:45   ` Lennart Borgman
2010-06-30 12:27     ` Agustin Martin
2010-06-30 17:35       ` Eli Zaretskii
2010-07-05 11:30         ` Agustin Martin
2010-07-05 11:52         ` Agustin Martin
2010-07-05 12:27           ` Lennart Borgman
2010-07-05 17:23           ` Eli Zaretskii [this message]
2010-07-05 17:32             ` Lennart Borgman
2010-07-05 17:39               ` Eli Zaretskii
2010-07-05 17:55                 ` Lennart Borgman
2010-07-05 18:23                   ` Eli Zaretskii
2010-07-05 19:50                     ` Lennart Borgman
2010-07-06 13:48                       ` Agustin Martin
2010-07-06 15:12             ` Agustin Martin
2010-07-06 18:26               ` Eli Zaretskii
2010-06-30 18:07       ` Lennart Borgman
2010-06-30 17:31     ` Eli Zaretskii
2010-06-30 18:13       ` Lennart Borgman
2010-06-30 20:55         ` Eli Zaretskii
2010-06-30 21:04           ` Lennart Borgman
2010-07-01  3:16             ` Eli Zaretskii
2010-07-01  9:45               ` Lennart Borgman
2010-07-01 16:54                 ` Eli Zaretskii
2010-07-01 17:01                   ` Lennart Borgman
2011-07-13 23:42                   ` Lars Magne Ingebrigtsen
2010-07-01  1:34   ` Stefan Monnier
2010-07-01  3:17     ` Eli Zaretskii
2010-07-23 15:32       ` Stefan Monnier
2010-07-23 16:47         ` Agustin Martin
2010-07-23 18:01         ` Eli Zaretskii

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=83mxu59716.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=6539@debbugs.gnu.org \
    --cc=agustin.martin@hispalinux.es \
    /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.