After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 619728 - EXTENSION_MAPPING for C# not working
EXTENSION_MAPPING for C# not working
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.6.3-SVN
Other All
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2010-05-26 15:26 UTC by Vsevolod Kukol
Modified: 2011-10-05 12:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to rename c# parser to csharp, so it gets recognized in EXTENSION_MAPPING (1.62 KB, patch)
2010-05-26 15:26 UTC, Vsevolod Kukol
none Details | Review
fix hardcoded checks for *.cs extension (1.28 KB, patch)
2010-05-26 15:27 UTC, Vsevolod Kukol
none Details | Review

Description Vsevolod Kukol 2010-05-26 15:26:27 UTC
Created attachment 162024 [details] [review]
patch to rename c# parser to csharp, so it gets recognized in EXTENSION_MAPPING

I'm working on a new VB.NET filter and wanted to generate C# code.
Therefor I had to map *.vb files to the C# parser, but it doesn't work.

Configuration:

EXTENSION_MAPPING      = .vb=C#
FILE_PATTERNS          = *.vb
OPTIMIZE_OUTPUT_JAVA   = YES
EXTRACT_ALL            = YES

With *.cs files renamed to *.vb Doxygen still uses the default C parser configuration.

After a closer look at the code, I've found several problems with this configuration.

1. EXTENSION_MAPPING = .vb=C# can't work, because the "#" is recognized as a comment.
I've patched the util.cpp to use "csharp" insted of "c#".

2. there are some lines with a hrdcoded detection of *.cs files.
docparser.cpp 4984
filedef.cpp   96   (BTW: line 95 for Java as well, bug 567774)
pre.l         134

applying attached patches solves this problem.
Comment 1 Vsevolod Kukol 2010-05-26 15:27:25 UTC
Created attachment 162025 [details] [review]
fix hardcoded checks for *.cs extension
Comment 2 Dimitri van Heesch 2010-06-04 16:02:01 UTC
Thanks for the reminder Vsevolod, I'll include your patches in the next release.
Comment 3 Dimitri van Heesch 2010-06-15 11:22:09 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.7.0. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information
that you think can be relevant.
Comment 4 Patrick Kursawe 2011-10-05 12:23:02 UTC
Please reopen this bug until you have fixed the documentation. There is not even a slight hint that you have to write "csharp" instead of "C#".
Comment 5 Patrick Kursawe 2011-10-05 12:31:52 UTC
(In reply to comment #4)
> Please reopen this bug until you have fixed the documentation. There is not
> even a slight hint that you have to write "csharp" instead of "C#".

To be more precise, config.html#cfg_extension_mapping gives no hint.
The help of the GUI frontend has a different problem: It lists C and C++ twice.