[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#61205: 'function' in 3rd element of treesit-font-lock-feature-list
From: |
Randy Taylor |
Subject: |
bug#61205: 'function' in 3rd element of treesit-font-lock-feature-list |
Date: |
Thu, 02 Feb 2023 03:18:07 +0000 |
On Wednesday, February 1st, 2023 at 21:34, Dmitry Gutov <dgutov@yandex.ru>
wrote:
> On 01/02/2023 07:18, Yuan Fu wrote:
>
> > > On Jan 31, 2023, at 6:08 PM, Dmitry Gutov dgutov@yandex.ru wrote:
> > >
> > > X-Debbugs-Cc: Yuan Fu casouri@gmail.com, Randy Taylor dev@rjt.dev
> > >
> > > Some new built-in modes has 'function' feature highlighting thus enabled
> > > by default.
> > >
> > > rust-ts-mode, go-ts-mode, cmake-mode
> > >
> > > Should we move it to 4 for consistency with the rest?
> > >
> > > Previously, we talked about that and concluded that function calls are
> > > usually everywhere and are easy to notice without additional highlighting.
> >
> > Right, I think they should be level 4.
>
>
> OK, I tried simply changing that, and it turned out none of these modes
> have a 'definition' feature, so highlightings get stripped from all
> definitions as well.
>
> And in a couple of cases function calls got highlighted either as a
> type, or as a property. I fixed the latter by deleting one selector, and
> the latter didn't touch per se.
>
> But since the same modes had 'variable' and 'property' features in the
> 3rd level as well, I moved them to the 4th (that resolved the incorrect
> highlighting as property mentioned above, but it's probably need to be
> improved later, in case someone will want to enable 'property' but not
> 'function' highlighting).
>
> And I added the 'definition' features and moved some highlighting rules
> there. And added some.
>
> So the patch looks a bit more complex than expected, see attached.
>
> cmake-ts-mode, in the end, I ended up keeping as-is. The 'variable'
> selector is more functional there than everywhere else (the grammar uses
> those nodes for template expansion), and if 'function' is removed, the
> buffer looks almost devoid of highlighting.
>
> Also none of these modes have highlighting for function parameters or
> assignments. That can be improved later.
I just took a quick look (will have more time tomorrow hopefully), but:
Why are there 2 separate definition features for both go and rust?
There should just be one each.
> - (scoped_identifier name: (identifier) @font-lock-type-face)
Why was this removed? I think this will lead to some imports not being
highlighted at the very least.
And please keep treesit-font-lock-feature-list alphabetized :).