Unnecessary function call in comment link

Minor performance issue. In comment.module (4.7 and 5.0) the function comment_link makes some unnecessary queries in cases where there are no comments. That is, there's no need to run comment_num_new if comment_num_all establishes that there aren't any comments on a node. (These two functions build the familiar "4 comments - 2 new comments" type links in teaser views.)

The problem is that comment_num_new in turn calls node_last_viewed, and then runs its own query to see if any of the node's comments are more recent than the user's last visit to the node.

if (user_access('access comments')) {
        $all = comment_num_all($node->nid);
        $new = comment_num_new($node->nid);
       

But if there aren't any comments, why bother checking to see if there are any _new_ comments?

      if (user_access('access comments')) {
        $all = comment_num_all($node->nid);
        if ($all) {
          $new = comment_num_new($node->nid);
        }

Trivial, but annoying to know that on a teaser page with 10 nodes, no comments, and links turned on, 20 extra db queries are being made...