GNU bug report logs - #67527
30.0.50; FR: Add an option to disable ispell completion in text-mode

Previous Next

Full log

Message #59 received at 67527 <at> debbugs.gnu.org (full text, mbox):

From: Daniel Mendler <mail <at> daniel-mendler.de>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 67527 <at> debbugs.gnu.org, aqua0210 <at> foxmail.com, me <at> eshelyaron.com,
 simon <at> ecksd.com
Subject: Re: bug#67527: 30.0.50; FR: Add an option to disable ispell
 completion in text-mode
Date: Sat, 20 Jan 2024 11:33:47 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Daniel Mendler <mail <at> daniel-mendler.de>
>> Cc: Eason Huang <aqua0210 <at> foxmail.com>,  Eshel Yaron <me <at> eshelyaron.com>,
>>   67527 <at> debbugs.gnu.org,  simon <at> ecksd.com
>> Date: Sat, 20 Jan 2024 10:46:00 +0100
>> 
>> Eli Zaretskii <eliz <at> gnu.org> writes:
>> 
>> > Eshel, given this feedback, I think it would make sense to add a new
>> > defcustom that would make text-mode avoid adding
>> > ispell-completion-at-point to completion-at-point-functions.  Would
>> > you please submit such a change and the documentation changes to go
>> > with it?  TIA.
>> 
>> Maybe the existing defcustom `text-mode-meta-tab-ispell-complete-word'
>> could be renamed and reused, taking three values? This way we avoid
>> introducing multiple defcustoms for tweaking the same feature.
>> 
>> - `command': Bind `ispell-complete-word' to TAB.
>> - `capf': Add `ispell-completion-at-point' to the
>>   `completion-at-point-functions'.
>> - `nil': None of the above.
>
> Reused, yes.  Renamed, I'd rather prefer we avoided that, as it causes
> compatibility problems.  If we can extend its list of valid values to
> support this aspect as well, it would be good.

Iiuc the variable was introduced a short while ago on Emacs master, so
renaming it may still be possible?

Daniel

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.