We are making loads of doxygen formatting errors in our source. Here is a patch for menu.inc which does proper unordered lists. For an example of the badness, see the list of node paths at http://api.drupal.org/api/function/menu_get_ancestors/6

CommentFileSizeAuthor
#3 mw3.patch3.46 KBpancho
mw.patch3.28 KBmoshe weitzman

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Needs work

The second hunk has the first list item buggy. The third hunk does not indent the list.

moshe weitzman’s picture

moshe weitzman commented
Assigned: moshe weitzman » Unassigned

Sorry about that. I don't have api module setup locally do i can't quickly test this. Hopefully someone else can fix this up, even in a bug fix release if needed.

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
StatusFileSize
new mw3.patch3.46 KB

@Gábor: Is this one good?
I corrected the first item of the one hunk. Also I further indented all list items, as it seemed to me more correct.
Didn't understand what you meant with "The third hunk does not indent the list.", though.

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
Status: Needs work » Needs review
ax’s picture

ax commented
Status: Needs review » Reviewed & tested by the community

tested on a local api.module install - works as advertised.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 6.x-dev » 7.x-dev

Thanks, committed #3, still RTBC for 7.x.

gpk’s picture

gpk commented

@moshe: cool, I've added a comment to the "Doxygen formatting conventions" page http://drupal.org/node/1354#comment-748664 about how to generate unordered lists.

Are there any other tricks like this e.g. ordered lists?

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.