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 736029 - [PATCH] Remove redundant if/else branch in src/lodepng.cpp
[PATCH] Remove redundant if/else branch in src/lodepng.cpp
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.8-GIT
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2014-09-04 10:21 UTC by Ettl Martin
Modified: 2014-12-25 16:02 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed fix (508 bytes, patch)
2014-09-04 10:21 UTC, Ettl Martin 		none Details | Review

Description Ettl Martin 2014-09-04 10:21:53 UTC 
Created attachment 285335 [details] [review]
proposed fix

Please review the attached patch. It simply removes an redundant if/else branch in src/lodepng.cpp.

Best regards and many thanks

Martin Ettl
Comment 1 Dimitri van Heesch 2014-09-09 21:03:14 UTC 
Thanks, I'll include the patch in the next GIT update.
Looks like also case 1 has the same code for if and else...
Comment 2 Dimitri van Heesch 2014-12-25 16:02:41 UTC 
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.9. 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 (preferrably in the form of a self-contained example).