GNU bug report logs - #72223
31.0.50; c-ts-mode-indent-style custom function should accept a `mode`(c/c++) parameter

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#72223: closed (31.0.50; c-ts-mode-indent-style custom
 function should accept a `mode`(c/c++) parameter )
Date: Sun, 21 Jul 2024 09:36:02 +0000

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

Your message dated Sun, 21 Jul 2024 12:35:38 +0300
with message-id <865xszcck5.fsf <at> gnu.org>
and subject line Re: bug#72223: 31.0.50; c-ts-mode-indent-style custom function should accept a `mode`(c/c++) parameter
has caused the debbugs.gnu.org bug report #72223,
regarding 31.0.50; c-ts-mode-indent-style custom function should accept a `mode`(c/c++) parameter 
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


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

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

From: Meow King <mr.meowking <at> anche.no>
To: bug-gnu-emacs <at> gnu.org
Subject: 31.0.50; c-ts-mode-indent-style custom function should accept a
 `mode`(c/c++) parameter 
Date: Sat, 20 Jul 2024 22:10:06 +0800

Currently, the `c-ts-mode--get-indent-style` function: 
```
(defun c-ts-mode--get-indent-style (mode)
  "Helper function to set indentation style.
MODE is either `c' or `cpp'."
  (let ((style
         (if (functionp c-ts-mode-indent-style)
             (funcall c-ts-mode-indent-style)
           (alist-get c-ts-mode-indent-style (c-ts-mode--indent-styles mode)))))
    `((,mode ,@style))))
```
We can use `c-ts-mode-indent-style` to customize our indentation.
However, the function doesn't accept a `mode` parameter, without which
we cannot determine the current customizing indentation style of what
language (c/c++).



---


In GNU Emacs 31.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version
3.24.42, cairo version 1.18.0)
Repository revision: 3a790abd869ddadc343710deb0c4368227ba6611
Repository branch: master
System Description: NixOS 24.11 (Vicuna)

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Meow King <mr.meowking <at> anche.no>
Cc: 72223-done <at> debbugs.gnu.org
Subject: Re: bug#72223: 31.0.50; c-ts-mode-indent-style custom function
 should accept a `mode`(c/c++) parameter
Date: Sun, 21 Jul 2024 12:35:38 +0300

> From: Meow King <mr.meowking <at> anche.no>
> Cc: 
> Date: Sun, 21 Jul 2024 17:14:35 +0800
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > Isn't the mode available as the value of major-mode?
> 
> Yes, after testing, this approach is proved to be right.
> 
> My `major-mode` related thoughts was immediately discarded since the
> function call is inside `define-derived-mode` block, and not even the
> `after-hook`. After testing, it is proved that the `major-mode` variable
> is already settled. Though, I think it's better to give it a mention in the doc.
> 
> Thank you for your reply and your awesome work!

Thanks, so I'm closing this bug report.

P.S. Please in the future use Reply All to reply, so that the bug
tracker is CC'ed.

Previous Next

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