Module for comment validation
In your form_set_error() call, you haven't set a specific field name on which to set an error. It's possible that without a form_set_error() call identifying a real form element no error is happening.

form_set_error('subject', t('Error!'));


Also, check to make sure your validation function is being called with a drupal_set_message()