[james-project] 35/44: [Refactoring] MessageResultImpl: headers are never null

[bt...@apache.org]

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 57da2c3ded75f9c1503a360eb57fb4d52dd8f59d
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Fri Nov 15 10:24:32 2019 +0700

    [Refactoring] MessageResultImpl: headers are never null
---
 .../main/java/org/apache/james/mailbox/store/MessageResultImpl.java  | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageResultImpl.java b/mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageResultImpl.java
index fb3b9cb..18cb8ea 100644
--- a/mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageResultImpl.java
+++ b/mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageResultImpl.java
@@ -54,9 +54,9 @@ public class MessageResultImpl implements MessageResult {
 
     private final Map<MimePath, PartContent> partsByPath = new HashMap<>();
     private final MailboxMessage message;
+    private final HeadersImpl headers;
 
     private MimeDescriptor mimeDescriptor;
-    private HeadersImpl headers;
     private Content fullContent;
     private Content bodyContent;
 
@@ -305,9 +305,6 @@ public class MessageResultImpl implements MessageResult {
     
     @Override
     public Headers getHeaders() {
-        if (headers == null) {
-            headers = new HeadersImpl(message);
-        }
         return headers;
     }
     


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