`M-q' uses tree-sitter now?

`M-q' uses tree-sitter now?

[Eduardo Ochs]

Hi list,

I am running an Emacs30 that I git-pulled and compiled a few hours ago
- its emacs-repository-version is

  "5a344d90c53d681ed98ab03ad3e218a3c6561108"

and I noticed that if I type `M-q' (`fill-paragraph') in a buffer in
emacs-lisp-mode I get this error:

  prog-fill-reindent-defun: Symbol's function definition is void:
  treesit-parser-list

I grepped the Lisp sources and `treesit-parser-list' is only mentioned
in two files: "treesit.el" and "progmodes/prog-mode.el". I tried

  (require 'prog-mode)
  (require 'treesit)

and `M-q' to see if the error would go away... but nah, same thing.
So I thought that this would be worth reporting...

  Cheers =/,
    Eduardo Ochs
    http://angg.twu.net/eepitch.html

Re: `M-q' uses tree-sitter now?

[Theodor Thornhill]

On 18 December 2022 14:44:25 CET, Eduardo Ochs <eduardoochs@gmail.com> wrote:
>Hi list,
>
>I am running an Emacs30 that I git-pulled and compiled a few hours ago
>- its emacs-repository-version is
>
>  "5a344d90c53d681ed98ab03ad3e218a3c6561108"
>
>and I noticed that if I type `M-q' (`fill-paragraph') in a buffer in
>emacs-lisp-mode I get this error:
>
>  prog-fill-reindent-defun: Symbol's function definition is void:
>  treesit-parser-list
>
>I grepped the Lisp sources and `treesit-parser-list' is only mentioned
>in two files: "treesit.el" and "progmodes/prog-mode.el". I tried
>
>  (require 'prog-mode)
>  (require 'treesit)
>
>and `M-q' to see if the error would go away... but nah, same thing.
>So I thought that this would be worth reporting...
>
>  Cheers =/,
>    Eduardo Ochs
>    http://angg.twu.net/eepitch.html
>

Strange. Shouldn't it be enough to put the require in the eval-when-compile? Is anything else needed for the function to be picked up?

Re: `M-q' uses tree-sitter now?

[Eli Zaretskii]

> Date: Sun, 18 Dec 2022 15:53:29 +0100
> From: Theodor Thornhill <theo@thornhill.no>
> 
> Strange. Shouldn't it be enough to put the require in the eval-when-compile? Is anything else needed for the function to be picked up?

His Emacs was built without tree-sitter, would be my guess.  We need
to call treesit-available-p when this can happen, before calling any
treesit functions that might not be fboundp.

Re: `M-q' uses tree-sitter now?

[Theodor Thornhill]

On 18 December 2022 18:16:57 CET, Eli Zaretskii <eliz@gnu.org> wrote:
>> Date: Sun, 18 Dec 2022 15:53:29 +0100
>> From: Theodor Thornhill <theo@thornhill.no>
>> 
>> Strange. Shouldn't it be enough to put the require in the eval-when-compile? Is anything else needed for the function to be picked up?
>
>His Emacs was built without tree-sitter, would be my guess.  We need
>to call treesit-available-p when this can happen, before calling any
>treesit functions that might not be fboundp.

Right, makes sense. I guess this can occur other places as well. I can look around and see if it's needed other places.

Theo

Re: `M-q' uses tree-sitter now?

[Eli Zaretskii]

> Date: Sun, 18 Dec 2022 19:08:30 +0100
> From: Theodor Thornhill <theo@thornhill.no>
> CC: emacs-devel@gnu.org, eduardoochs@gmail.com
> 
> 
> 
> On 18 December 2022 18:16:57 CET, Eli Zaretskii <eliz@gnu.org> wrote:
> >> Date: Sun, 18 Dec 2022 15:53:29 +0100
> >> From: Theodor Thornhill <theo@thornhill.no>
> >> 
> >> Strange. Shouldn't it be enough to put the require in the eval-when-compile? Is anything else needed for the function to be picked up?
> >
> >His Emacs was built without tree-sitter, would be my guess.  We need
> >to call treesit-available-p when this can happen, before calling any
> >treesit functions that might not be fboundp.
> 
> Right, makes sense. I guess this can occur other places as well. I can look around and see if it's needed other places.

Please do, and thanks.

Re: `M-q' uses tree-sitter now?

[Eli Zaretskii]

> From: Eduardo Ochs <eduardoochs@gmail.com>
> Date: Sun, 18 Dec 2022 10:44:25 -0300
> 
> Hi list,
> 
> I am running an Emacs30 that I git-pulled and compiled a few hours ago
> - its emacs-repository-version is
> 
>   "5a344d90c53d681ed98ab03ad3e218a3c6561108"
> 
> and I noticed that if I type `M-q' (`fill-paragraph') in a buffer in
> emacs-lisp-mode I get this error:
> 
>   prog-fill-reindent-defun: Symbol's function definition is void:
>   treesit-parser-list

Thanks, I tried to fix this now.  Please test.

P.S. And _please_ always report unexpected signals and other obvious
problems as bugs to the bug tracker, not here.

Re: `M-q' uses tree-sitter now?

[Eduardo Ochs]

On Sun, 18 Dec 2022 at 14:15, Eli Zaretskii <eliz@gnu.org> wrote:
>
> > From: Eduardo Ochs <eduardoochs@gmail.com>
> > Date: Sun, 18 Dec 2022 10:44:25 -0300
> >
> > Hi list,
> >
> > I am running an Emacs30 that I git-pulled and compiled a few hours ago
> > - its emacs-repository-version is
> >
> >   "5a344d90c53d681ed98ab03ad3e218a3c6561108"
> >
> > and I noticed that if I type `M-q' (`fill-paragraph') in a buffer in
> > emacs-lisp-mode I get this error:
> >
> >   prog-fill-reindent-defun: Symbol's function definition is void:
> >   treesit-parser-list
>
> Thanks, I tried to fix this now.  Please test.
>
> P.S. And _please_ always report unexpected signals and other obvious
> problems as bugs to the bug tracker, not here.

The error is gone!
Sure, I will use the bug tracker the next time...
Sorry for the noise in emacs-devel!
Thanks,
  Eduardo