menu_link_save has formatting issues

Thanks for reporting this. It was fixed in the Drupal 7 version, but not in Drupal 6.

Needs to have
@see l
changed to
see l()
on that line of the documentation, so that l() will turn into a link to the l() function automatically. http://api.drupal.org/api/function/l

A good project for a novice drupal contributor...

Here you go.

Oh. Actually now that I look at the line you patched, it should not have @see there at all. We don't put @see inside a @param or @return section, because it starts a completely new See Also section. It should just be the word See. Can you fix that too?

Updated with your suggestions.

Beh, updating case.

Sorry for the delay in reviewing...

The list in this function param is not formatted according to guidelines:
http://drupal.org/node/1354#lists

I am reluctant to mark this patch RTBC since the function doc being fixed is out of compliance with our standards...

No worries, I'm learning.

Thanks... That's closer, but the list is still not compliant. List items should look like:

- item_name: Description starts with capital letter, ends with period.

And there should be a blank line between the end of the @param and the @return.

Try, try again. Sorry I'm missing these and thanks for sticking it out with me.

that's better, thanks!

This is definitely an improvement on the existing docs, and fixes the reported problem. Let's get it in...

This patch is fine. The test bot is acting strange and saying "can't find file to patch at input line 8"

Committed, thank you.