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

[aduprat]

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 <btell...@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-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org