You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by bt...@apache.org on 2021/06/26 05:06:20 UTC

[james-project] 02/06: [PERFORMANCE] SimpleMailboxMessage: userFlags assignment was done twice

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

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

commit c2dedfe865d0c62641d115e55d49a70acb34fc9f
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Mon Jun 21 13:16:46 2021 +0700

    [PERFORMANCE] SimpleMailboxMessage: userFlags assignment was done twice
    
    Non negligible as a copy of the underlying vector was performed...
---
 .../apache/james/mailbox/store/mail/model/impl/SimpleMailboxMessage.java | 1 -
 1 file changed, 1 deletion(-)

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 7d8ee37..ef2aaba 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
@@ -206,7 +206,6 @@ public class SimpleMailboxMessage extends DelegatingMailboxMessage {
 
             setFlags(flags);
             this.mailboxId = mailboxId;
-            this.userFlags = flags.getUserFlags();
     }
 
     public SimpleMailboxMessage(MessageId messageId, Date internalDate, long size, int bodyStartOctet,

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