Improper use of t()

In the code there are examples of t() being used like:

  t('Override ' . $name . ' setting')

This really isn't all that good for translators. Instead it should be something like:

  t('Override !name setting', array('!name' => $name))

This allows the translators to put the variable where ever it is appropriate in the text.

You can find more detail at http://api.drupal.org/api/function/t/6