From: Vincenzo Pupillo <v.pupillo@gmail.com>
To: 71518@debbugs.gnu.org
Cc: eliz@gnu.org, casouri@gmail.com
Subject: bug#71518: 30.0.50; Errors during redisplay with c++-ts-mode
Date: Thu, 13 Jun 2024 10:57:50 +0200 [thread overview]
Message-ID: <2461821.9o76ZdvQCi@3-191.divsi.unimi.it> (raw)
In-Reply-To: <7108109B-3AFB-4623-890E-1E19903FA0D0@gmail.com>
In data giovedì 13 giugno 2024 08:51:21 CEST, Yuan Fu ha scritto:
>
> > On Jun 12, 2024, at 6:53 AM, Vincenzo Pupillo <vincenzo.pupillo@lpsd.it> wrote:
> >
> >
> > In data mercoledì 12 giugno 2024 14:33:17 CEST, Eli Zaretskii ha scritto:
> >> To reproduce in "emacs -Q", visit some C++ source file (I used
> >> breakpoint.c from the GDB sources), then turn on c++-ts-mode. The
> >> fontification is all incorrect, and *Messages* has these error
> >> messages:
> >>
> >> Error during redisplay: (jit-lock-function 7286) signaled (treesit-query-error "Node type error at" 575 "[\"_Atomic\" \"break\" \"case\" \"const\" \"continue\" \"default\" \"do\" \"else\" \"enum\" \"extern\" \"for\" \"goto\" \"if\" \"inline\" \"register\" \"restrict\" \"return\" \"sizeof\" \"static\" \"struct\" \"switch\" \"typedef\" \"union\" \"volatile\" \"while\" \"and\" \"and_eq\" \"bitand\" \"bitor\" \"catch\" \"class\" \"co_await\" \"co_return\" \"co_yield\" \"compl\" \"concept\" \"consteval\" \"constexpr\" \"constinit\" \"decltype\" \"delete\" \"explicit\" \"final\" \"friend\" \"mutable\" \"namespace\" \"new\" \"noexcept\" \"not\" \"not_eq\" \"operator\" \"or\" \"or_eq\" \"override\" \"private\" \"protected\" \"public\" \"requires\" \"template\" \"throw\" \"try\" \"typename\" \"using\" \"virtual\" \"xor\" \"xor_eq\"] @font-lock-keyword-face (auto) @font-lock-keyword-face (this) @font-lock-keyword-face" "Debug the query with `treesit-query-validate'")
> >>
> >> This happens with the latest C++ tree-sitter grammar library. I guess
> >> they made some incompatible change in the grammar or something?
> >>
> >>
> > Yes, they changed the grammar.
> > https://github.com/tree-sitter/tree-sitter-cpp/issues/271
> >
> > V.
>
> So it seems “virtual” is now a named node rather than an anonymous node, which is a little odd. I asked the author about it in the GitHub issue, let me clear that up before fixing c-ts-mode.
>
> And, Eli, we really really should pin tree-sitter grammars. As more people start using tree-sitter modes, they will found out about the breakage before we do and the tree-sitter major mode would be unusable for them before they upgrade their grammar. Most of them will not know how to do it and which commit to downgrade to. And I really don’t want to chase the random changes these grammars make all the time. Fixing for these breakages is arguably a heavier maintenance burden than pinning tree-sitter grammars.
+1.
This is why php-ts-mode has a feature just for installing specific versions of grammars (unfortunately phpdoc has no tags on the repository :-( )
Vincenzo
>
> Yuan
>
>
>
next prev parent reply other threads:[~2024-06-13 8:57 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-12 12:33 bug#71518: 30.0.50; Errors during redisplay with c++-ts-mode Eli Zaretskii
2024-06-12 13:53 ` Vincenzo Pupillo
2024-06-13 6:51 ` Yuan Fu
2024-06-13 8:24 ` Eli Zaretskii
2024-06-13 8:57 ` Vincenzo Pupillo [this message]
2024-06-22 8:26 ` Eli Zaretskii
2024-06-22 14:42 ` Vincenzo Pupillo
2024-06-22 15:57 ` Eli Zaretskii
2024-06-22 16:14 ` Vincenzo Pupillo
2024-06-22 16:56 ` Eli Zaretskii
2024-06-22 17:03 ` Eli Zaretskii
2024-06-22 17:59 ` Vincenzo Pupillo
2024-06-22 18:15 ` Eli Zaretskii
2024-06-22 18:15 ` Vincenzo Pupillo
2024-06-22 18:19 ` Eli Zaretskii
2024-06-22 23:18 ` Yuan Fu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2461821.9o76ZdvQCi@3-191.divsi.unimi.it \
--to=v.pupillo@gmail.com \
--cc=71518@debbugs.gnu.org \
--cc=casouri@gmail.com \
--cc=eliz@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).