Coding hook_uninstall(): Perform node_delete()s to keep TABLE {node} garbage free?

Hi

TOPIC: Implementing a hook_uninstall()
REFERENCE: Custom module example: node_example_uninstall

PROBLEM?: When calling a hook_uninstall(), it does NOT call implicitly any of the following:

1. node_delete()
2. hook_delete()
3. hook_nodeapi:$op= delete
4. hook_nodeapi:$op= delete revision

This surprises me!

Since none of the aforementioned calls are made during an uninstall, then this leads to TABLE {node} filling with deprecated (i.e. garbage) entries because they are no longer referred to by custom module's schema.

For example, node_example_schema maintains both vid and nid as a reference to an entry in the TABLE {node}.

When TABLE {node_example} is reinstalled, then the old node entries in the TABLE {node} that were previously reference by the entries of TABLE {node_example} prior to uninstall are all basically garbage. Am I wrong here?

SOLUTION?: Upon hook_uninstall(), should not one recursively call node_delete() on all its node references within their custom module's schema BEFORE calling drupal_uninstall_schema()

Should not node_example_uninstall really be implemented as follows to keep TABLE {node} garbage free:

function node_example_uninstall() {
  $result = db_query( 'SELECT nid FROM {node_example}' );
  while($row = db_fetch_array($query_result)) {
    $nid = $row['nid']
    node_delete($nid);
  }

  drupal_uninstall_schema('node_example');
}

If not, then why not?

Thanks

Jeff in Seattle