GNU bug report logs - #63390
29.0.90; c-ts-mode fails to recognize functions in xterm.c

Previous Next

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Yuan Fu <casouri <at> gmail.com>
Cc: 63390 <at> debbugs.gnu.org
Subject: bug#63390: 29.0.90; c-ts-mode fails to recognize functions in xterm.c
Date: Mon, 15 May 2023 14:13:47 +0300

> From: Yuan Fu <casouri <at> gmail.com>
> Date: Sun, 14 May 2023 22:47:06 -0700
> Cc: 63390 <at> debbugs.gnu.org
> 
> >>  (function_declarator declarator: (identifier)
> >>   parameters: 
> >>    (parameter_list (
> >>     (parameter_declaration
> >>      type: (struct_specifier struct name: (type_identifier))
> >>      declarator: (pointer_declarator * declarator: (identifier)))
> >>     )))
> >> 
> >> Same with the next function, x_draw_composite_glyph_string_foreground.
> >> But the function after that, x_draw_glyphless_glyph_string_foreground,
> >> is again recognized as function definition.  I wonder if the
> >> preprocessor conditionals around there have something to do with that.
> 
> Ok, so that’s because there are ifdef’s inside the function, which cuts the function into pieces and tree-sitter can’t make out a function_definition, which is what we use to fontify the function name. A function_declarator alone can be used in many places, like in an argument list for function pointers, I think?

Does this mean that on master movement by defuns will be broken around
those functions?

> I can fix this by fontifying top-level function_declaration, I think a top-level function_declaration should always be a function definition?

Yes, I think this would be better.

> >> Btw, function declarations in a header file are recognized as such,
> >> but the names of the functions there are still correctly fontified.
> 
> They are fine because there’s a semicolon in the end, so the function_decalration is wrapped in a declaration node, which we (the fontification rules) recognize.

Thanks for explaining this.

Previous Next

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