flyspell-auto-correct-word many wrong-type-argument

flyspell-auto-correct-word many wrong-type-argument

[Dan Jacobson]

Today I happened to have debug-on-error t, and saw
Debugger entered--Lisp error: (wrong-type-argument vectorp nil)
  define-abbrev(nil "hittng" "hitting")
  flyspell-auto-correct-word()
  call-interactively(flyspell-auto-correct-word)

It does correct the word though.
http://jidanni.org/comp/emacs.txt is my .emacs file.

Another:
Debugger entered--Lisp error: (wrong-type-argument vectorp nil)
  define-abbrev(nil "happpen" "happen")...

Re: flyspell-auto-correct-word many wrong-type-argument

[Kevin Rodgers]

Dan Jacobson wrote:

> Today I happened to have debug-on-error t, and saw
> Debugger entered--Lisp error: (wrong-type-argument vectorp nil)
>   define-abbrev(nil "hittng" "hitting")
>   flyspell-auto-correct-word()
>   call-interactively(flyspell-auto-correct-word)
> 
> It does correct the word though.
> http://jidanni.org/comp/emacs.txt is my .emacs file.
> 
> Another:
> Debugger entered--Lisp error: (wrong-type-argument vectorp nil)
>   define-abbrev(nil "happpen" "happen")...


What is the value of flyspell-use-global-abbrev-table-p?


What was the buffer's major mode?

-- 
Kevin Rodgers

Re: flyspell-auto-correct-word many wrong-type-argument

[Dan Jacobson]

Sometimes flyspell-auto-correct-word does its job when esc tab is hit, but
complains in the minibuffer "Wrong type argument: vectorp: nil".

>>>>> "K" == Kevin Rodgers <ihs_4664@yahoo.com> writes:

K> What is the value of flyspell-use-global-abbrev-table-p ?

nil

K> What was the buffer's major mode?

Message mode.

8 lines matching " minor mode" in buffer *Help*.
      2:For minor modes, see following pages.
    157:Mml minor mode (indicator MML):
    159:MML is the MIME Meta Language, a minor mode for composing MIME articles.
    201:Mc-Write minor mode (indicator MC-w):
    215:Flyspell minor mode (indicator Fly):
    220:With a prefix argument ARG, turn Flyspell minor mode on iff ARG is positive.
    244:Abbrev minor mode (indicator Abbrev):
    250:auto-fill-function minor mode (indicator Fill):

in fact it happens as I compose this message too, so add SC minor mode
(supercite) to the above.

Wait, if I (setq flyspell-use-global-abbrev-table-p t) the problem
goes away.

I see, I will do this in .emacs to hush up the problem, while the
elves will please fix it behind the scenes.

Re: flyspell-auto-correct-word many wrong-type-argument

[Kevin Rodgers]

Dan Jacobson wrote:

> Sometimes flyspell-auto-correct-word does its job when esc tab is hit, but
> complains in the minibuffer "Wrong type argument: vectorp: nil".
> 
> 
>>>>>>"K" == Kevin Rodgers <ihs_4664@yahoo.com> writes:
>>>>>>
> 
> K> What is the value of flyspell-use-global-abbrev-table-p ?
> 
> nil
> 
> K> What was the buffer's major mode?
> 
> Message mode.
> 
> 8 lines matching " minor mode" in buffer *Help*.
>       2:For minor modes, see following pages.
>     157:Mml minor mode (indicator MML):
>     159:MML is the MIME Meta Language, a minor mode for composing MIME articles.
>     201:Mc-Write minor mode (indicator MC-w):
>     215:Flyspell minor mode (indicator Fly):
>     220:With a prefix argument ARG, turn Flyspell minor mode on iff ARG is positive.
>     244:Abbrev minor mode (indicator Abbrev):
>     250:auto-fill-function minor mode (indicator Fill):
> 
> in fact it happens as I compose this message too, so add SC minor mode
> (supercite) to the above.
> 
> Wait, if I (setq flyspell-use-global-abbrev-table-p t) the problem
> goes away.
> 
> I see, I will do this in .emacs to hush up the problem, while the
> elves will please fix it behind the scenes.

The problem is here:


(defun flyspell-abbrev-table ()
   (if flyspell-use-global-abbrev-table-p
       global-abbrev-table
     local-abbrev-table))


Since local-abbrev-table can be nil, flyspell-abbrev-table can return nil.
Should its callers check for that, or should it be changed to return an
abbrev table:

(defun flyspell-abbrev-table ()
   (cond (flyspell-use-global-abbrev-table-p global-abbrev-table)
         (local-abbrev-table)
         (t (setq local-abbrev-table (make-abbrev-table)))))

-- 
Kevin Rodgers

Re: flyspell-auto-correct-word many wrong-type-argument

[Richard Stallman]

Does this replacement function give good results?

(defun flyspell-abbrev-table ()
  (if flyspell-use-global-abbrev-table-p
      global-abbrev-table
    (or local-abbrev-table global-abbrev-table)))

Re: flyspell-auto-correct-word many wrong-type-argument

[Kevin Rodgers]

Richard Stallman wrote:

> Does this replacement function give good results?
> 
> (defun flyspell-abbrev-table ()
>   (if flyspell-use-global-abbrev-table-p
>       global-abbrev-table
>     (or local-abbrev-table global-abbrev-table)))

It prevents the error, but has the undesirable effect of using the
global abbrev table when flyspell-use-global-abbrev-table-p is nil.


-- 
Kevin Rodgers