Unnecessary string tranlation for the node title

Problem/Motivation

In the print_mail module, the node title is translated with the t()-function. I think this is unnecessary an fills only the translate table.

if (!empty($title)) {
    if (drupal_substr($path, 0, 5) == 'node/') {
      $title = t($title);
    }

    $form['fld_title'] = array(
      '#type' => 'item',
      '#title' => t('Page to be sent'),
      '#value' => '<span id="sent-title">'. l($title, $path, array('attributes' => array('title' => t('View page')))) .'</span>',
      '#id' => 'sent-title',
    );
  }

Maybe it's a good idea to translate other other stings, when is not a node (e.g. a view title).

Proposed resolution

Only translate the title when it's not a node-title.