GNU bug report logs - #60127
treesit-end-of-defun: possible bug with clojure grammar

Previous Next

Package: emacs;

Reported by: Danny Freeman <Danny <at> dfreeman.email>

Date: Fri, 16 Dec 2022 18:53:01 UTC

Severity: normal

Done: Stefan Kangas <stefankangas <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Yuan Fu <casouri <at> gmail.com>
Cc: 60127 <at> debbugs.gnu.org, danny <at> dfreeman.email, dgutov <at> yandex.ru
Subject: Re: bug#60127: treesit-end-of-defun: possible bug with clojure grammar
Date: Tue, 17 Jan 2023 14:20:43 +0200
> From: Yuan Fu <casouri <at> gmail.com>
> Date: Tue, 17 Jan 2023 01:30:22 -0800
> Cc: 60127 <at> debbugs.gnu.org,
>  Danny Freeman <danny <at> dfreeman.email>,
>  Eli Zaretskii <eliz <at> gnu.org>
> 
> 
> > I’d rather not use treesit_search_dfs, as it does much more work than
> > the current code. You know what, I can write a custom
> > ts_node_first_child_for_byte and use that in Emacs, I’ve already done
> > that for ts_node_parent anyway.
> 
> I did that. I tested with the Clojure example and it works fine now.
> 
> Eli, we don’t use ts_node_first_child_for_pos for now until tree-sitter
> fix that function, should I removed the boilerplate for it or should I
> leave it as-is? I saw you removed ts_node_parent when I did a similar
> fix.

It should be either removed or #ifdef'ed away, because otherwise GCC
complains in the MS-Windows build about unused macro.




This bug report was last modified 1 year and 259 days ago.

Previous Next


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