From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Lennart Borgman Newsgroups: gmane.emacs.bugs Subject: bug#6539: ispell-complete-word looks for wrong file (at least on w32) Date: Mon, 5 Jul 2010 14:27:50 +0200 Message-ID: References: <20100630103826.GA3847@agmartin.aq.upm.es> <20100630122747.GA20074@agmartin.aq.upm.es> <836310bezc.fsf@gnu.org> <20100705115222.GA2013@agmartin.aq.upm.es> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Trace: dough.gmane.org 1278333002 26211 80.91.229.12 (5 Jul 2010 12:30:02 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Mon, 5 Jul 2010 12:30:02 +0000 (UTC) Cc: 6539@debbugs.gnu.org To: Agustin Martin Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Jul 05 14:30:00 2010 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1OVko4-0000RA-AJ for geb-bug-gnu-emacs@m.gmane.org; Mon, 05 Jul 2010 14:29:52 +0200 Original-Received: from localhost ([127.0.0.1]:36300 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OVko3-0006x4-Jj for geb-bug-gnu-emacs@m.gmane.org; Mon, 05 Jul 2010 08:29:51 -0400 Original-Received: from [140.186.70.92] (port=54655 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OVknx-0006wI-O2 for bug-gnu-emacs@gnu.org; Mon, 05 Jul 2010 08:29:46 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OVknw-0000A1-Ep for bug-gnu-emacs@gnu.org; Mon, 05 Jul 2010 08:29:45 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:36925) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OVknw-00009x-Cg for bug-gnu-emacs@gnu.org; Mon, 05 Jul 2010 08:29:44 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.69) (envelope-from ) id 1OVknG-0002T4-5A; Mon, 05 Jul 2010 08:29:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lennart Borgman Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-To: owner@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 05 Jul 2010 12:29:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 6539 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 6539-submit@debbugs.gnu.org id=B6539.12783328989475 (code B ref 6539); Mon, 05 Jul 2010 12:29:02 +0000 Original-Received: (at 6539) by debbugs.gnu.org; 5 Jul 2010 12:28:18 +0000 Original-Received: from localhost ([127.0.0.1] helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OVkmX-0002Sm-Vs for submit@debbugs.gnu.org; Mon, 05 Jul 2010 08:28:18 -0400 Original-Received: from mail-wy0-f172.google.com ([74.125.82.172]) by debbugs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OVkmV-0002Sh-RT for 6539@debbugs.gnu.org; Mon, 05 Jul 2010 08:28:16 -0400 Original-Received: by wyg36 with SMTP id 36so1677451wyg.3 for <6539@debbugs.gnu.org>; Mon, 05 Jul 2010 05:28:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:received:in-reply-to :references:from:date:message-id:subject:to:cc:content-type :content-transfer-encoding; bh=atZ7EOEAhkwPsbRk2bqYTvrqCZSTo/yKyce/mjE9jrs=; b=Ohijf00OlpVg4LMal2gg2KCfH09GOwiED33tZ/5pC6hWxfNOPaK4b3nbS7DwtVchdJ 2y1POWzqKR5vMZvT8TSypJx1Ta/QhhZ811il2dziQYiJ6A52HA/0i1ayWzUgTbqkSMAT 41nAag+ivh067fPHYUHzEobkHUKXBhbdZPagY= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type:content-transfer-encoding; b=UH4tFFwtKgm0IUwT8OHyFUWW4HFBQUePkKbiAg5FeKMnwdV/1qg0OaBf3RQXUrtULD PNZVgvxna+//gp33OK2qPmQS0KAjTBiPe1Nbfd5wyQI/ebbYsk/xS2G28roy2w1mISMV VZoAP9H3YHU96Xtuvj4R5jzHYkMkqSkU+4DAQ= Original-Received: by 10.213.26.84 with SMTP id d20mr2112429ebc.43.1278332890238; Mon, 05 Jul 2010 05:28:10 -0700 (PDT) Original-Received: by 10.213.15.132 with HTTP; Mon, 5 Jul 2010 05:27:50 -0700 (PDT) In-Reply-To: <20100705115222.GA2013@agmartin.aq.upm.es> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.11 Precedence: list Resent-Date: Mon, 05 Jul 2010 08:29:02 -0400 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 3) X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:38235 Archived-At: On Mon, Jul 5, 2010 at 1:52 PM, Agustin Martin wrote: >> > >> > Or, more globally, find a good and not confusing name for default pers= onal >> > plain wordlist that can be used in every system, be it w32 or not, so = it is >> > tried first in `ispell-alternate-dictionary', something like >> > >> > (expand-file-name "~/.personal.words") >> >> How will that help, exactly? =C2=A0/usr/dict/words exists on almost ever= y >> Posix system, which is why ispell.el has that as the default. =C2=A0But >> there's no comparable functionality on Windows, so the only way of >> getting this work on Windows is to find or download such a file from >> somewhere, and manually install it in some arbitrary directory. =C2=A0Ho= w >> is Emacs supposed to know whether such a file exists, and where? > > [Really attaching diff] > > My initial intention was to provide an out-of-the box default for persona= l > wordlist, useful for both w32 and *X systems. However, as you point out i= n > another message, since this is originally an Ispell feature, I think I > should not add an Emacs specific difference. Customizing > `ispell-alternate-dictionary' (or `ispell-complete-word-dict') seems the > way to go. > > 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. > > It also changes `file-exists-p' to `file-readable-p' in > `ispell-alternate-dictionary' definition. > > I have also noticed that ispell.el uses `ispell-alternate-dictionary' and > `ispell-complete-word-dict' in a way that seems to be interchangeable, bu= t > confusing. The former was probably used at some time as a real alternate > dictionary for spellchecking, but current use seems to be for completion. > Have to think a bit more about this. Thanks Agustin, but I still think it is a very bad idea to have unix-style file names as default for other systems.