API page: http://api.drupal.org/api/drupal/includes%21module.inc/function/module_i...
The gist of this comment http://api.drupal.org/comment/35778#comment-35778 should be added to the docs for both functions.
Sounds like a good thing to document that arguments are not passed by reference. Should probably also mention that if you need to pass things by reference, you can use drupal_alter() (or alter hooks in general).
This patch documents that arguments are passed by value and refers to drupal_alter() as an alternative.
Thanks! I'll get this committed shortly unless one of the other maintainers gets to it first.
Thanks! Committed to 8.x. Needs porting to 7.x.
D7 patch attached.
Thanks! Committed to 7.x.
Automatically closed -- issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.