Logic for displaying "X new comment(s)" link seems inverted

I don't know how this could go unnoticed this whole time, but I am positive that the logic for adding the 'comment-new-comments' link in comment_node_view() is actually the wrong way round:

$new = comment_num_new($node->nid);
if (!$new) {
  $links['comment-new-comments'] = array(
    'title' => format_plural($new, '1 new comment', '@count new comments'),
    // …

comment_num_new() will return 0 if there are no new comments or FALSE if the user isn't logged in – resulting, for me, in links "0 new comments" and " new comments" being displayed.
However, if the user is logged in and there are new comments, no such link is displayed. And the code above seems to agree.

If this really is the case (and I didn't overlook something):
- this has been round since at least late Alphas
- as said, I can't possibly imagine how this could go unnoticed for such a long time
- we definitely shouldn't ship with this, as it would be really embarassing – making this major, I think
- the attached patch would fix this

But probably I just overlooked something again and will (or, rather, this issue will) be banished into the dark "closed (works as designed)" realm …