JAMES-1854 Use Optional to better write MailDispatcher Builder
Project: http://git-wip-us.apache.org/repos/asf/james-project/repo Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/09c522be Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/09c522be Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/09c522be Branch: refs/heads/master Commit: 09c522befecd5ffc795ed33ff584c6552ee2ea96 Parents: 42800ae Author: Benoit Tellier <[email protected]> Authored: Thu Nov 10 15:58:32 2016 +0700 Committer: Benoit Tellier <[email protected]> Committed: Fri Nov 18 18:46:46 2016 +0700 ---------------------------------------------------------------------- .../james/transport/mailets/delivery/MailDispatcher.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/james-project/blob/09c522be/server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/delivery/MailDispatcher.java ---------------------------------------------------------------------- diff --git a/server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/delivery/MailDispatcher.java b/server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/delivery/MailDispatcher.java index 3bcd662..04968a6 100644 --- a/server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/delivery/MailDispatcher.java +++ b/server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/delivery/MailDispatcher.java @@ -45,13 +45,14 @@ public class MailDispatcher { } public static class Builder { + public static final boolean CONSUME_DEFAULT_VALUE = true; private MailStore mailStore; - private boolean consume; + private Optional<Boolean> consume = Optional.absent(); private MailetContext mailetContext; private Log log; public Builder consume(boolean consume) { - this.consume = consume; + this.consume = Optional.of(consume); return this; } @@ -74,7 +75,7 @@ public class MailDispatcher { Preconditions.checkNotNull(mailStore); Preconditions.checkNotNull(log); Preconditions.checkNotNull(mailetContext); - return new MailDispatcher(mailStore, consume, log, mailetContext); + return new MailDispatcher(mailStore, consume.or(CONSUME_DEFAULT_VALUE), log, mailetContext); } } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
