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 701550 - Error List window in Visual Studio no longer recognizes doxygen warnings
Error List window in Visual Studio no longer recognizes doxygen warnings
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: build
1.8.4
Other Windows
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2013-06-03 18:22 UTC by Robert
Modified: 2013-08-23 15:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add WARN_STRING and ERROR_STRING config options (1.72 KB, patch)
2013-06-03 20:54 UTC, Robert
none Details | Review

Description Robert 2013-06-03 18:22:22 UTC
Hello.

I have setup a custom tool in Visual Studio that runs doxygen in the Output Window. When warnings appear there, they are conveniently displayed also in the Error List window. The warning messages changed in 1.8.4. Now "Warning:" is capitalized, whereas in 1.8.3 it was lowercase "warning:". Because of this change in case, the Error List no longer recognizes it. The MSVC compiler users lowercase "warning", so the Error List must be hardcoded to be case sensitive.

Please allow me to customize the text for warning in the doxygen config. Default value can be "Warning:", but it will allow me to specify lowercase "warning:" so that Doxygen warnings will be processed by other tools properly.

Example in config:

WARNING_STRING = "warning:"

Thanks.
Comment 1 Robert 2013-06-03 20:54:59 UTC
Created attachment 245961 [details] [review]
Add WARN_STRING and ERROR_STRING config options

I have implemented this for you, it was pretty easy to do. I tested doxygen with this change against the code base at work and seems to work perfectly. I didn't update doxywizard or anything other than doxygen.exe. If you provide some guidance I can do that for you but I am not as familiar with the code as you are. Let me know if you like the solution.
Comment 2 Philipp Moeller 2013-06-03 21:14:31 UTC
This also broke some of my tools, but I feel like making those things configurable would be overkill. Doxygen should simply produce warnings and errors that can be parsed by Visual Studio, e.g. lower case "warning" and "error".
Comment 3 Robert 2013-06-03 21:20:29 UTC
Philipp, I think this is a bit overkill as well but consider that while Visual Studio expects lowercase, what if some other IDE expects uppercase? In that case we can never have both tools working. The only way to make them both work is to make it configurable :(

I had more general extensibility in mind, not just Visual Studio. What are your thoughts on this?
Comment 4 Philipp Moeller 2013-06-03 21:55:21 UTC
All IDEs I work with have customizable error output scanners. It comes as a surprise to me that VS doesn't support this and that its scanner is that naive.
Comment 5 Robert 2013-06-03 21:59:01 UTC
I agree Philipp, I tried to find some hidden documentation on possible registry settings to configure or something of that sort but I've had no luck :(
Comment 6 Dimitri van Heesch 2013-06-04 17:57:46 UTC
I will change the defaults to lower case. I see also gcc uses lower case.
This avoids additional options, and will still work for tools that have the necessary flexibility.
Comment 7 Dimitri van Heesch 2013-08-23 15:04:39 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.5. 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.