opendev/gerrit
Code Issues Proposed changes

Merge "Log when appropriate if email delivery is skipped"

Browse Source
This commit is contained in:
Shawn Pearce
2013-04-22 17:41:51 +00:00
committed by Gerrit Code Review
parent ec82f0de4f 9cf8301e6d
commit 3b5934866d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java
View File

@@ -268,7 +268,7 @@ public abstract class OutgoingEmail {
protected boolean shouldSendMessage() { protected boolean shouldSendMessage() {
if (body.length() == 0) { if (body.length() == 0) {
// If we have no message body, don't send. // If we have no message body, don't send.
// log.warn("Skipping delivery of email with no body");
return false; return false;
} }
@@ -276,7 +276,7 @@ public abstract class OutgoingEmail {
// If we have nobody to send this message to, then all of our // If we have nobody to send this message to, then all of our
// selection filters previously for this type of message were // selection filters previously for this type of message were
// unable to match a destination. Don't bother sending it. // unable to match a destination. Don't bother sending it.
// log.info("Skipping delivery of email with no recipients");
return false; return false;
} }