GNU bug report logs - #73404
30.0.50; [forward/kill/etc]-sexp commands do not behave as expected in tree-sitter modes

Previous Next

Full log

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

From: Juri Linkov <juri <at> linkov.net>
To: Yuan Fu <casouri <at> gmail.com>
Cc: Mickey Petersen <mickey <at> masteringemacs.org>, Eli Zaretskii <eliz <at> gnu.org>,
 Theodor Thornhill <theo <at> thornhill.no>, 73404 <at> debbugs.gnu.org,
 Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#73404: 30.0.50; [forward/kill/etc]-sexp commands do not
 behave as expected in tree-sitter modes
Date: Mon, 30 Dec 2024 09:15:42 +0200

> Also added 'sexp-list' to c-ts-mode, js-ts-mode, ruby-ts-mode and
> html-ts-mode.  Addition of 'sexp-list' to other ts-modes is underway.

BTW, my initial intention was to add the thing 'list'.
But then I discovered that (treesit-thing-next (point) 'list)
uses the function 'list' instead of the thing 'list'.

However, the current 'sexp-list' as a two-word composite is too ugly.
Now I found a better replacement: 'group'.  This word is already used
in lisp.el such as in the docstring of 'forward-list':

  "Move forward across one balanced group of parentheses."

And the error messages: "No next group".

So exactly the same message will be used by

  (format-message "No next %S" pred)

where 'pred' will be 'group'.  IMHO, this looks better:

  (setq-local treesit-thing-settings
              `((html
                 (sexp ,(regexp-opt '("element" "text" "attribute" "value")))
                 (group ,(regexp-opt '("element"")))
                 (sentence "tag")
                 (text ,(regexp-opt '("comment" "text"))))))

Previous Next

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