GNU bug report logs - #72116
30.0.60; Filling c-ts-mode documentation is broken

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Damien Cassou <damien <at> cassou.me>
Subject: bug#72116: closed (Re: bug#72116: 30.0.60; Filling c-ts-mode
 documentation is broken)
Date: Wed, 17 Jul 2024 04:20:01 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#72116: 30.0.60; Filling c-ts-mode documentation is broken

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 72116 <at> debbugs.gnu.org.

-- 
72116: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72116
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Yuan Fu <casouri <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Damien Cassou <damien <at> cassou.me>, 72116-done <at> debbugs.gnu.org
Subject: Re: bug#72116: 30.0.60; Filling c-ts-mode documentation is broken
Date: Tue, 16 Jul 2024 21:18:27 -0700

> On Jul 14, 2024, at 11:44 AM, Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
>> From: Damien Cassou <damien <at> cassou.me>
>> Date: Sun, 14 Jul 2024 20:23:09 +0200
>> 
>> Filling documentation in a c-ts-mode buffer doesn't insert the " * "
>> prefix. It works fine in c-mode. How to reproduce:
>> 
>> 1. Install the C treesitter grammar
>> 2. Start emacs with "emacs -Q"
>> 3. Open the attached file in a new buffer
>> 4. Run M-x c-ts-mode
>> 5. Move point to the middle of line 2
>> 6. Press M-q
> 
> You can work around this by enabling auto-fill-mode.  Then typing SPC
> at the end of the long line will produce the effect you want.
> 
> Yuan, can you please look into this?

Fixed and pushed to master. I didn’t push this to emacs-30 since adaptive-filling is a fragile being, so let’s wait and see :-)

> 
>> If you remove step 4 from the above instructions, c-mode will do the
>> work just fine.
> 
> In c-mode, M-q is bound to a c-mode specific function, which makes no
> sens in c-ts-mode.  We need a ts-specific solution (and we already
> seem to have it, it just isn't working in this case for some reason).

In c-ts-mode, M-q jumps a few hops, but it eventually calls c-ts-mode’s filling function (c-ts-common--fill-paragraph).

Yuan

[Message part 3 (message/rfc822, inline)]

From: Damien Cassou <damien <at> cassou.me>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.60; Filling c-ts-mode documentation is broken
Date: Sun, 14 Jul 2024 20:23:09 +0200

[Message part 4 (text/plain, inline)]

Filling documentation in a c-ts-mode buffer doesn't insert the " * "
prefix. It works fine in c-mode. How to reproduce:

1. Install the C treesitter grammar
2. Start emacs with "emacs -Q"
3. Open the attached file in a new buffer
4. Run M-x c-ts-mode
5. Move point to the middle of line 2
6. Press M-q

Expected: 2 lines of documentation, each one starting with " * "

Actual: 2 lines of documentation, only the first one starting
with " * "

If you remove step 4 from the above instructions, c-mode will do the
work just fine.

The solution to this bug could help solve bug#71760.

-- 
Damien Cassou

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill

[foo.c (text/plain, attachment)]

Previous Next

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