Subject: bug#60122: 29.0.60; c-ts-mode breaks with '"' separated macros

From: Theodor Thornhill <[email-masked]>
Date: Fri, 16 Dec 2022 20:30:39 +0100

Mohammed Sadiq <[email-masked]> writes:

> The following code is parsed wrong with c-ts-mode:
>
>
> #include <gio/gio.h>
>
> int
> main (void)
> {
>   GFileInfo *info;
>   GFile *file;
>
>   file = g_file_new_for_path ("some/path");
>   info = g_file_query_info (file,
>                             G_FILE_ATTRIBUTE_THUMBNAIL_IS_VALID ","
>                             G_FILE_ATTRIBUTE_THUMBNAILING_FAILED ","
>                             G_FILE_ATTRIBUTE_TIME_MODIFIED ","
>                             G_FILE_ATTRIBUTE_STANDARD_FAST_CONTENT_TYPE,
>                             G_FILE_QUERY_INFO_NONE,
>                             NULL, NULL);
> }
>
>
> c-ts-mode doesn't assume  G_FILE_ATTRIBUTE_THUMBNAIL_IS_VALID (and
> others) to be a string (or some function/macro call that results in a
> string) resulting in broken font locking and tab indentation.
>
> The same code seems to work fine with c-mode
>

Hi!

If you use treesit-explore-mode, you can see that the tree-sitter parser
returns this ast:

   (expression_statement
    (assignment_expression left: (identifier) operator: =
     right: 
      (call_expression function: (identifier)
       arguments: 
        (argument_list ( (identifier) , (identifier)
         (ERROR ")
         ,
         (string_literal "
          (ERROR (identifier))
          ")
         ,
         (string_literal "
          (ERROR (identifier))
          ")
         ,
         (ERROR ")
         (identifier) , (identifier) , (null) , (null) ))))
    ;)

Because tree-sitter fails to parse this there's only so much we can do.

I think you should file this as a bug at the parser issue tracker over
at github[0].  Maybe it's a known problem already?

Theo

[0]: https://github.com/tree-sitter/tree-sitter-c/issues