Question about invoking hook_flag

I'm working on a module that needs to save a bit of extra data anytime a node is flagged or unflagged so I was looking at how you handle the list flagging in your code. This seems to be the relevant block:

  // Perform the flagging or unflagging of this flag. We invoke hook_flag here
  // because we do our own flagging.
  $flagged = _flag_lists_is_flagged($flag, $entity_id, $uid, $sid);
  if ($action == 'unflag') {
    if ($flagged) {
      $fcid = _flag_lists_unflag($flag, $entity_id, $uid, $sid);
      module_invoke_all('flag', 'unflag', $flag, $entity_id, $account, $fcid);
      return TRUE;
    }
  }
  elseif ($action == 'flag') {
    if (!$flagged) {
      $fcid = _flag_lists_flag($flag, $entity_id, $uid, $sid);
      module_invoke_all('flag', $flag, $entity_id, $account, $fcid);
      return TRUE;
    }
  }

I'm not sure if this is deliberate. But in the first instance you invoke module_invoke_all('flag', 'unflag', $flag, $entity_id, $account, $fcid);, setting the $op parameter to "unflag". In the second though, there is no $op parameter.

Also, in the flag 3.x API which I think the latest version of flag_lists supports they've replaced hook_flag with hook_flag_flag and hook_flag_unflag. I'd be happy to write a patch for this if it's not deliberate.

Cheers,
- Scott