Code

notify_email plugin: Remove useless function and fix indentation.
authorFlorian Forster <octo@collectd.org>
Tue, 16 Aug 2011 06:50:40 +0000 (08:50 +0200)
committerFlorian Forster <octo@collectd.org>
Tue, 16 Aug 2011 06:50:40 +0000 (08:50 +0200)
src/notify_email.c

index 08f865fc2c57805866ce3475a3ec9b3a77ed0e9a..6b4678a7882a22baa010a569ba06cf1e7993ab6b 100644 (file)
@@ -298,14 +298,11 @@ static int notify_email_notification (const notification_t *n,
     return (-1);
   } else {
     #if COLLECT_DEBUG
-      const smtp_status_t *status;
-      /* Report on the success or otherwise of the mail transfer. */
-      status = smtp_message_transfer_status (message);
-      DEBUG ("notify_email plugin: SMTP server report: %d %s",
-        status->code, (status->text != NULL) ? status->text : "\n");
-    #else
-      //I don't know if the function below has side affects so i'm calling it to be on the safe side.
-      smtp_message_transfer_status (message);
+    const smtp_status_t *status;
+    /* Report on the success or otherwise of the mail transfer. */
+    status = smtp_message_transfer_status (message);
+    DEBUG ("notify_email plugin: SMTP server report: %d %s",
+      status->code, (status->text != NULL) ? status->text : "\n");
     #endif
     smtp_enumerate_recipients (message, print_recipient_status, NULL);
   }