search

emacs-languagetool / flymake-languagetool

Flymake support for LanguageTool
GNU General Public License v3.0
33 stars 11 forks source link

flymake-languagetool-server-command should be of type list #28

Closed telenieko closed 2 months ago

telenieko commented 2 months ago

Hi,

https://github.com/emacs-languagetool/flymake-languagetool/blob/e52307cdd7b1708171b32e6f928bc482e1237a83/flymake-languagetool.el#L109-L113

flymake-languagetool-server-command is defined as string, but later on flymake-languagetool--start-server it is used in place of a list:

https://github.com/emacs-languagetool/flymake-languagetool/blob/e52307cdd7b1708171b32e6f928bc482e1237a83/flymake-languagetool.el#L392-L396

which results in

Debugger entered--Lisp error: (wrong-type-argument stringp 108)
  flymake-languagetool--start-server(nil)
jcs090218 commented 2 months ago

You should assign it to a list. 🤔

telenieko commented 2 months ago

My bad, didn't realise that :type '(repeat string) means "a list of strings". !!