Add $module and $hook as formal parameters in module_invoke()

Issue Summary

Problem/Motivation

The doxygen function documentation for module_invoke() lists both $hook and $module as parameters, but these are currently retrieved with func_get_args() rather than being declared in the function definition. Adding the parameters to the function definitions would make them more self-documenting and would also provide more meaningful error messages when the correct parameters are not passed.

The same is true of module_invoke_all() for its $hook argument.

Proposed resolution

Patch in #1216758-4: Add $module and $hook as formal parameters in module_invoke() adds the arguments to the function definitions. So:
module_invoke()
becomes
module_invoke($module, $hook)

and
module_invoke_all()
becomes
module_invoke_all($hook)

Remaining tasks

None. The $module and $hook parameters are already documented in the functions' docblocks.

User interface changes

None.

API changes

The documented signatures for module_invoke() and module_invoke_all() would technically change, but there would be no functional change since both use func_get_args(). (These functions already trigger errors when the $hook or $module arguments are not passed.)

This change could be backported to D7 and even D6.

Original report by @tim.plunkett

Looking at module_invoke, I don't understand why $module and $hook aren't in the function definition.

I'm expecting there is a reason why this isn't done, but if so, can we document why it is done this way?

This would also make things break cleaner if someone tried to call module_invoke() alone.