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 625296 - Too much <td></td> within html dev.memitem div.memproto table.memname.
Too much <td></td> within html dev.memitem div.memproto table.memname.
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.7.1
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2010-07-26 10:03 UTC by Stefan Oberhumer
Modified: 2010-10-09 08:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Sample output before and after fix. (38.14 KB, image/png)
2010-07-26 10:03 UTC, Stefan Oberhumer
  Details
Sample project to create the screenshot (3.43 KB, application/x-zip)
2010-07-26 10:06 UTC, Stefan Oberhumer
  Details
Patch: Fix too much <td></td> within html dev.memitem div.memproto table.memname. (1.59 KB, patch)
2010-07-26 10:07 UTC, Stefan Oberhumer
none Details | Review

Description Stefan Oberhumer 2010-07-26 10:03:18 UTC
Created attachment 166568 [details]
Sample output before and after fix.

Within html export the detailed function description contains trailing td's at the last two lines.
This fix which applies to doxygen-1.7.1 corrects the problem.

Without this fix:
  <table>
  <tr><td>int foo</td><td>(</td><td>int</td><td>param1,</td></tr>
  <tr><td></td>       <td></td> <td>int</td><td>param2,</td><td></td></tr>
  <tr><td></td>       <td>)</td><td>int</td><td></td>       <td></td></tr>

With fix:
  <table>
  <tr><td>int foo</td><td>(</td><td>int</td><td>param1,</td></tr>
  <tr><td></td>       <td></td> <td>int</td><td>param2,</td></tr>
  <tr><td></td>       <td>)</td><td>int</td><td></td>       </tr>
Comment 1 Stefan Oberhumer 2010-07-26 10:06:33 UTC
Created attachment 166569 [details]
Sample project to create the screenshot
Comment 2 Stefan Oberhumer 2010-07-26 10:07:41 UTC
Created attachment 166570 [details] [review]
Patch: Fix too much <td></td> within html dev.memitem div.memproto table.memname.
Comment 3 Dimitri van Heesch 2010-08-01 14:32:48 UTC
Thanks for the patch. Should be included in the next subversion update.
Comment 4 Dimitri van Heesch 2010-10-09 08:19:17 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.7.2. 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.