hook_uninstall API page needs to be more detailed

I think it needs some more detailed wording. How about:

Drupal 5

Remove any information that the module sets itself. This includes:

• variables the module has set using variable_set
• variables the module can set indirectly in its forms, like the settings form
• blocks that the module defines
• tables the module created

The module should not remove its entry from the {system] table.

The uninstall hook will fire when the module gets uninstalled.

Drupal 6

Remove any information that the module sets itself. This includes:

• variables the module has set using variable_set
• variables the module can set indirectly in its forms, like the settings form
• blocks that the module defines
• the module schema

The module should not remove its entry from the {system] table.

The uninstall hook will fire when the module gets uninstalled.

See for instance http://drupal.org/node/211175 about these practices.