hook_nodequeue_delete should provide an object, not a qid as parameter

I think you're probably correct that this is a better way, but this will break modules already implementing hook_nodequeue_delete(), including smartqueue.module. I wonder if it might be better to introduce the object as an additional (optional) argument.

function nodequeue_delete($qid) {
  $queue = nodequeue_load($qid);
   ...
  module_invoke_all('nodequeue_delete', $qid, $queue);
}

// ----------------------------
/**
 * Implements hook_nodequeue_delete().
 */
function foo_nodequeue_delete($qid) {
  $args = func_get_args();
  $queue = $args[1];
  ...
}

@othermachines, good call. The new hook is now module_invoke_all('nodequeue_delete', $qid, $queue);