watchdogs wrong

I believe that according to http://api.drupal.org/api/function/watchdog/6 the watchdog calls in FAQ_ASK are wrong (Drupal5-format not Drupal6).

See example (faq_ask.module, line 393ff - latest from CVS)

      if ($mail_sent) {
         watchdog('FAQ_Ask', t('Expert notification email sent.') .' '. $to, WATCHDOG_NOTICE);
      }

should perhaps be

      if ($mail_sent) {
         watchdog('FAQ_Ask', t('Expert notification email sent.') .' '. $to, null, WATCHDOG_NOTICE);
      }

The above change does not incorporate the possibility of watchdog translating automatically, though the call to t() could be avoided:

      if ($mail_sent) {
         watchdog('FAQ_Ask', t('Expert notification email sent. %to'), array('%to'=>$to), WATCHDOG_NOTICE);
      }

Related problems can be found throughout the code.

BEGIN UNRELATED RANT
I would agree that it is a *** API-change, as a compatible API-change would have been possible without any loss, damage or additional work; changing an API just to change it is kind of ..
e.g. use watchdog($type, $message, $severity = WATCHDOG_NOTICE, $link = NULL, $variables=array())
This is one of the reasons why PHP supports optional parameters.
END UNRELATED RANT