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

Previous Next

Package: emacs;

Reported by: Mickey Petersen <mickey <at> masteringemacs.org>

Date: Sat, 21 Sep 2024 05:13:01 UTC

Severity: normal

Merged with 74366

Found in version 30.0.50

Fixed in version 31.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


Message #271 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: Theodor Thornhill <theo <at> thornhill.no>, Eli Zaretskii <eliz <at> gnu.org>,
 Mickey Petersen <mickey <at> masteringemacs.org>,
 Stefan Monnier <monnier <at> iro.umontreal.ca>, 73404 <at> debbugs.gnu.org
Subject: Re: bug#73404: 30.0.50; [forward/kill/etc]-sexp commands do not
 behave as expected in tree-sitter modes
Date: Fri, 10 Jan 2025 09:34:08 +0200
close 73404 31.0.50
thanks

>>> Juri, please feel free to apply your symbol property patch. For the symbol
>>> property name, I feel that something like
>>> treesit-symbol-predicate/treesit-use-as-symbol-predicate would be more
>>> descriptive.
>> 
>> If we settle on the variant that keeps the current format with symbols,
>> wouldn't it better to keep a symbol property name shorter?
>> 
>> First I proposed 'treesit-predicate', but then realized it's wrong
>> since it's opposite to the predicate function.
>> 'treesit-thing' was better since it indicates that it's used in
>> 'treesit-thing-settings'.
>> 
>>> Since we don’t have docstrings for symbol properties (right?).
>> 
>> There are no docstrings for symbol properties indeed.  But I think
>> the important property for the name is that it should be unique enough
>> to be easily greppable.  But I tried to grep, and 'treesit-thing' finds
>> too many matches.
>> 
>> So probably the most suitable symbol name would be 'treesit-thing-symbol',
>> that currently has no occurrences in source code.
>
> Sounds good. Let’s go with it then.

So now pushed together with updates in the manual.




This bug report was last modified 131 days ago.

Previous Next


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