Redundant token processing

When retrieving the rendered values for tokens on a field, the method that allows you to add additional tokens is called once for every field in the view definition, which is just wasted processing.

From views_handler_field:

  function get_render_tokens($item) {
    ...

    // Now add replacements for our fields.
    foreach ($this->view->display_handler->get_handlers('field') as $field => $handler) {
      if (isset($handler->last_render)) {
        $tokens["[$field]"] = $handler->last_render;
      }
      else {
        $tokens["[$field]"] = '';
      }
      $this->add_self_tokens($tokens, $item);   // <-- this should be outside the loop

      // We only use fields up to (and including) this one.
      if ($field == $this->options['id']) {
        break;
      }
    }
    ...
  }

Granted, work on work on Views seems to be dedicated to Views 3, but since the upgrade path from 2 to 3 still isn't worked out, I figured I'd toss this out there.