GNU bug report logs - #72814
31.0.50; Add a variable controlling doxygen support in C/C++/Java?

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#72814: closed (31.0.50; Add a variable controlling doxygen
 support in C/C++/Java?)
Date: Sat, 21 Sep 2024 09:45:03 +0000

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

Your message dated Sat, 21 Sep 2024 12:43:40 +0300
with message-id <86ikup4a0j.fsf <at> gnu.org>
and subject line Re: bug#72814: 31.0.50; Add a variable controlling doxygen support in C/C++/Java?
has caused the debbugs.gnu.org bug report #72814,
regarding 31.0.50; Add a variable controlling doxygen support in C/C++/Java?
to be marked as done.

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


-- 
72814: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72814
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: Bug Report Emacs <bug-gnu-emacs <at> gnu.org>
Subject: 31.0.50; Add a variable controlling doxygen support in C/C++/Java?
Date: Sun, 25 Aug 2024 20:56:31 -0700

X-Debbugs-CC: eliz <at> gnu.org <mailto:eliz <at> gnu.org>, vincenzo.pupillo <at> unimi.it

Should we add a custom option that controls whether to enable doxygen
support in c-ts-mode/c++-ts-mode/java-ts-mode?

Technically this isn’t a problem, since the doxygen support is only
enabled if there’s doxygen grammar on the system. But many people
(me) might install a grammar bundle that includes dozens of
grammars for convenience. Then the doxygen support would turn on when
the user doesn’t really intend to use it.

I propose we add custom options like c-ts-mode-enable-doxygen and set it
to t by default, so the default behavior is still to enable doxygen
support when the grammar for it exists, but people who don’t want to pay
for it can turn it off by setting the option to nil.

Yuan

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Vincenzo Pupillo <v.pupillo <at> gmail.com>
Cc: casouri <at> gmail.com, 72814-done <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#72814: 31.0.50;
 Add a variable controlling doxygen support in C/C++/Java?
Date: Sat, 21 Sep 2024 12:43:40 +0300

> From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
> Cc: bug-gnu-emacs <at> gnu.org, 72814 <at> debbugs.gnu.org
> Date: Sat, 14 Sep 2024 19:26:07 +0200
> 
> > > Another idea: use tertiary value for c-ts-mode-enable-doxygen, t for
> > > enable (and issue warning if doxygen isn’t present), nil for enable if
> > > doxygen exists but don’t issue any warning, ‘disabled for disable. (I
> > > don’t think there’s a convention for the three values of tertiary toggle,
> > > is that right Eli?)
> > That could also be OK, but I prefer something that does TRT by
> > default.
> 
> Ciao,
> in this new version of the patch, as default, the *-ts-mode-enable-doxygen are 
> set to nil, and issue a diagnostic (one per session) when Not-nil and if the  
> grammar isn't present. It is OK?

Thanks, installed on master, and closing the bug.

Previous Next

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