GNU bug report logs - #75362
29.4; go-ts-mode.el: wrong indentation inside var list

Previous Next

Package: emacs;

Reported by: Roi Martin <jroi.martin <at> gmail.com>

Date: Sat, 4 Jan 2025 17:47:02 UTC

Severity: normal

Found in version 29.4

Done: Yuan Fu <casouri <at> gmail.com>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Yuan Fu <casouri <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 75362 <at> debbugs.gnu.org, Randy Taylor <dev <at> rjt.dev>, jroi.martin <at> gmail.com
Subject: bug#75362: 29.4; go-ts-mode.el: wrong indentation inside var list
Date: Thu, 9 Jan 2025 19:11:14 -0800


> On Jan 8, 2025, at 11:10 PM, Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
>> Date: Wed, 08 Jan 2025 13:20:19 +0000
>> From: Randy Taylor <dev <at> rjt.dev>
>> Cc: Yuan Fu <casouri <at> gmail.com>, 75362 <at> debbugs.gnu.org, jroi.martin <at> gmail.com
>> 
>> On Wednesday, January 8th, 2025 at 08:02, Eli Zaretskii <eliz <at> gnu.org> wrote:
>>> 
>>> 
>>>> Cc: 75362 <at> debbugs.gnu.org, Roi Martin jroi.martin <at> gmail.com
>>> 
>>>> From: Yuan Fu casouri <at> gmail.com
>>>> Date: Tue, 7 Jan 2025 22:18:53 -0800
>>>> 
>>>>> The attached patch fixes it for me.<0001-Fix-go-ts-mode-var-spec-indentation-Bug-75362.patch>
>>>> 
>>>> Thanks Randy. Eli, should we apply this to emacs-30.
>>> 
>>> 
>>> Is this backward-compatible to older grammars? If yes, it's okay to
>>> install on emacs-30. If it is not compatible, I'd like to see how a
>>> compatible patch will look before I decide.
>> 
>> Yes, it's backward-compatible.
> 
> OK, then, Yuan, please feel free to install on emacs-30.

Installed.

As for cmake-ts-mode’s fix, it’s in 4def541bbfe67b1a492e8a3b6041a1d8f76d680b

Basically, tree-sitter-cmake gives up on parsing cmake generator expressions. According to the commit message:

> feat!: remove parsing generator expressions
> 
> Parsing generator expressions statically is impossible since whether an argument is interpreted as a generator expression or not depends on the function itself. Hence, the parsing for it is removed.
> 

Which makes sense to me. The fix is backward-compatible too, since we’re just removing fontification for angle brackets.

Yuan
This bug report was last modified 207 days ago.

Previous Next

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