Deleting children deletes all parent references

It seems that the code tries to make stuff a lot more robust than it need be:

/**
 * NodeAPI: called when a node is updated. This is just a wrapper around
 * clipper_insert, with an extra deletion of parent relationships for $node.
 */
function clipper_update($node) {
  db_query("DELETE FROM {relations} WHERE (right_id = %d AND relationship = 'parent-child')", $node->nid);
  if ($node->clipper_parent) {
    clipper_insert($node);
  }
}

/**
 * NodeAPI: called whenever a node is deleted.
 */
function clipper_delete($node) {
  if ($node->clipper->parents) {
    foreach ($node->clipper->parents as $parent) {
      db_query("DELETE FROM {relations} WHERE (left_id = %d AND right_id = %d AND relationship = 'parent-child')", $parent, $node->nid);
    }
  }
}

I think I agree with Allie Micka on the fact that the old clipper module was quite useful and to some extend I am tempted to replicate it and call it a heirarchy clipper module which only allows for clipping a child to a parent so [1:m] relationships... that seems a lot more useful to me ATM... this obviously just being IMO and not being an argument against this work... it's just a lot more complicated, for my purposes, than it need be...

Create a parent, then create many children. Now go and delete one of the children. I believe that should do it, let me know if it doesn't. The deletion queries to me seem like they're deleting things that this module doesn't yet create, looking at the insert queries.