GNU bug report logs - #62825
29.0.90; c-ts-mode doesn't recognize DEFUN functions

Previous Next

Package: emacs;

Reported by: Eli Zaretskii <eliz <at> gnu.org>

Date: Fri, 14 Apr 2023 06:45:01 UTC

Severity: normal

Found in version 29.0.90

Full log


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

From: Yuan Fu <casouri <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: theo <at> thornhill.no, 62825 <at> debbugs.gnu.org
Subject: Re: bug#62825: 29.0.90; c-ts-mode doesn't recognize DEFUN functions
Date: Sat, 15 Apr 2023 22:37:02 -0700
Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Yuan Fu <casouri <at> gmail.com>
>> Date: Fri, 14 Apr 2023 13:19:41 -0700
>> Cc: Theodor Thornhill <theo <at> thornhill.no>,
>>  62825 <at> debbugs.gnu.org
>> 
>> Ok, I fixed indent-defund and add-log. Because a DEFUN is made of
>> two nodes in the parse tree rather than one, it’s a bit hard to
>> incorporate it into the normal treesit-defun-at-point, etc,
>> functions. And we probably have to fix DEFUN case-by-case.
>
> Thanks.  But I wonder why do we need the c-ts-mode-emacs-devel knob.
> Why not recognize DEFUN by default everywhere?  CC Mode does that,
> AFAIK, so why shouldn't c-ts-mode do the same?

I just thought the fix is a bit hacky, and does add some overhead, so
people who don’t need it might not be happy to know that they can’t turn
it off.

Yuan




This bug report was last modified 2 years and 62 days ago.

Previous Next


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