[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#61403: 30.0.50; C tree-sitter bug?
From: |
Po Lu |
Subject: |
bug#61403: 30.0.50; C tree-sitter bug? |
Date: |
Sun, 12 Feb 2023 16:36:10 +0800 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Yuan Fu <casouri@gmail.com> writes:
> Po Lu <luangruo@yahoo.com> writes:
>
>> Go to sfnt.c in the feature/android branch, and turn on c-ts-mode.
>>
>> Then, go to line 10754, around which should be the function definition:
>>
>> /* Load the simple glyph GLYPH into the specified INTERPRETER, scaling
>> it up by INTERPRETER's scale, and run its glyph program if
>> present. Use the unscaled metrics specified in METRICS.
>>
>> Upon success, return NULL and the resulting points and contours in
>> *VALUE. Else, value is the reason interpretation failed. */
>>
>> TEST_STATIC const char *
>> sfnt_interpret_simple_glyph (struct sfnt_glyph *glyph,
>> struct sfnt_interpreter *interpreter,
>> struct sfnt_glyph_metrics *metrics,
>> struct sfnt_instructed_outline **value)
>> {
>> size_t zone_size, temp, outline_size, i;
>> struct sfnt_interpreter_zone *zone;
>> struct sfnt_interpreter_zone *volatile preserved_zone;
>> sfnt_f26dot6 phantom_point_1_x;
>>
>> `TEST_STATIC' is fontified as a type. Perhaps tree-sitter needs
>> something along the lines of `c-noise-macros'?
>
> Would it be reasonable to assume that all caps "type" are almost always
> macros? If it is, we can optionally defontify these kind of "types".
Not really, because an extremely common type is:
PTR_T *ptr;
where ``PTR_T'' is used to represent pointers on systems that may lack
properly working pointers to void.
The standard library FILE * is also one such type with a capitalized
name.
>>
>> Likewise for _Noreturn:
>>
>> _Noreturn static void
>> sfnt_interpret_trap (struct sfnt_interpreter *interpreter,
>> const char *reason)
>> {
>>
>> _Noreturn is a keyword in 2011 Standard C. I think the tree-sitter
>> parser definition files should be updated to understand it.
>
> Makes sense. I can file an issue on tree-sitter-c’s repo.
Please do so, and thanks.
- bug#61403: 30.0.50; C tree-sitter bug?, Po Lu, 2023/02/10
- bug#61403: 30.0.50; C tree-sitter bug?, Yuan Fu, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?,
Po Lu <=
- bug#61403: 30.0.50; C tree-sitter bug?, Eli Zaretskii, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Po Lu, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Eli Zaretskii, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Po Lu, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Eli Zaretskii, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Po Lu, 2023/02/12
- bug#61403: 30.0.50; C tree-sitter bug?, Eli Zaretskii, 2023/02/12