You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ad...@apache.org on 2019/07/04 09:29:54 UTC

[james-project] 07/15: JAMES-2816 Fix Intelij warnings in SimpleMailboxMessage

This is an automated email from the ASF dual-hosted git repository.

aduprat pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 398594db2e6869c37d8d65357ff82b1f3c3ab60b
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Mon Feb 18 10:12:45 2019 +0700

    JAMES-2816 Fix Intelij warnings in SimpleMailboxMessage
---
 .../mailbox/store/mail/model/impl/SimpleMailboxMessage.java    | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/mailbox/store/src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMailboxMessage.java b/mailbox/store/src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMailboxMessage.java
index 479053d..34b3b5c 100644
--- a/mailbox/store/src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMailboxMessage.java
+++ b/mailbox/store/src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMailboxMessage.java
@@ -138,13 +138,9 @@ public class SimpleMailboxMessage extends DelegatingMailboxMessage {
             SimpleMailboxMessage simpleMailboxMessage = new SimpleMailboxMessage(messageId, internalDate, size,
                 bodyStartOctet, content, flags, propertyBuilder, mailboxId, attachments.build());
 
-            if (uid.isPresent()) {
-                simpleMailboxMessage.setUid(uid.get());
-            }
+            uid.ifPresent(simpleMailboxMessage::setUid);
+            modseq.ifPresent(simpleMailboxMessage::setModSeq);
 
-            if (modseq.isPresent()) {
-                simpleMailboxMessage.setModSeq(modseq.get());
-            }
             return simpleMailboxMessage;
         }
     }
@@ -218,7 +214,7 @@ public class SimpleMailboxMessage extends DelegatingMailboxMessage {
                                 PropertyBuilder propertyBuilder, MailboxId mailboxId) {
         this(messageId, internalDate, size, bodyStartOctet,
                 content, flags,
-                propertyBuilder, mailboxId, ImmutableList.<MessageAttachment>of());
+                propertyBuilder, mailboxId, ImmutableList.of());
     }
 
     @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org