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

Previous Next

Package: emacs;

Reported by: Yuan Fu <casouri <at> gmail.com>

Date: Mon, 26 Aug 2024 03:58:01 UTC

Severity: normal

Merged with 72815

Found in version 31.0.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
To: casouri <at> gmail.com, 72814 <at> debbugs.gnu.org
Cc: eliz <at> gnu.org, vincenzo.pupillo <at> unimi.it
Subject: bug#72814: 31.0.50; Add a variable controlling doxygen support  in C/C++/Java?
Date: Mon, 09 Sep 2024 11:50:59 +0200
[Message part 1 (text/plain, inline)]
In data martedì 27 agosto 2024 14:22:16 CEST, Eli Zaretskii ha scritto:
> > From: Yuan Fu <casouri <at> gmail.com>
> > Date: Mon, 26 Aug 2024 20:13:52 -0700
> > Cc: eliz <at> gnu.org,
> > 
> >  vincenzo.pupillo <at> unimi.it
> > 
> > Yuan Fu <casouri <at> gmail.com> writes:
> > > 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
> > 
> > Eli, WDYT?
> 
> I'm okay with such an option, but it should be careful to check
> whether the corresponding grammar library is available, and offer a
> user-friendly diagnostic if not.

Try to take a look at this patch. Is it okay?

Vincenzo
[0001-Add-a-user-option-to-disable-Doxygen-syntax-highligh.patch (text/x-patch, attachment)]

This bug report was last modified 243 days ago.

Previous Next


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