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: Yuan Fu <casouri <at> gmail.com>
Subject: bug#72815: closed (Re: bug#72814: 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 bug report

#72814: 31.0.50; Add a variable controlling doxygen support in C/C++/Java?

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 72815 <at> debbugs.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: 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.

[Message part 3 (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

Previous Next

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