Re: "Can't find Hunspell dictionary" error (was Re: Emacs pretest 28.0.91 is out)

[Robert Pluim]

>>>>> On Thu, 13 Jan 2022 20:51:50 +1300, Phil Sainty <psainty@orcon.net.nz> 
>>>>> said:

    Phil> Eli Zaretskii <eliz@gnu.org> wrote:
    >> > From: Robert Pluim <rpluim@gmail.com>
    >> > Debugger entered--Lisp error: (error "Can't find Hunspell dictionary 
with a
    >> > .aff affix f...")
    >> >   signal(error ("Can't find Hunspell dictionary with a .aff affix 
f..."))
    >> >   error("Can't find Hunspell dictionary with a .aff affix f...")
    >> >   ispell-find-hunspell-dictionaries()
    >> >   ispell-set-spellchecker-params()
    >> >   ispell-valid-dictionary-list()
    >> >
    >> > This is because I have no hunspell dictionaries at all
    >> So you want to fix this?  I don't mind, but since it's (AFAIU)
    >> something specific to your installation, is it important to fix on
    >> the release branch?

    Phil> While unexpected, it doesn't seem *invalid* to have no dictionaries
    Phil> installed, so I think it's weird that `ispell-valid-dictionary-list'
    Phil> triggers (indirectly) an error in this scenario to essentially say
    Phil> "there were no valid dictionaries", when that's also what a nil return
    Phil> value would have meant.

    Phil> `ispell-valid-dictionary-list' or `ispell-set-spellchecker-params'
    Phil> should perhaps use `with-demoted-errors' to avoid this?

Perhaps

    Phil> Failing that, I think guarding against this in spelling-tests.el with
    Phil> `ignore-errors' makes sense.  It's an :unstable test, but it shouldn't
    Phil> be triggering actual errors.

Itʼs not the test thatʼs triggering errors, itʼs the check to see if
the test should be defined at all. Iʼll take a look at fixing that and
ispell-valid-dictionary-list on master.

Robert
--