[MAILBOX-179] Optimizes SimpleMessage::getFullContent()

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.3
• Fix Version/s: 0.4
• Component/s: store
• Environment: n/a
• Assignee: Eric Charles
• Reporter: Timothy Prepscius
• Created: 04/Jun/12 13:37
• Updated: 07/Jun/12 08:50
• Resolved: 07/Jun/12 08:50

Description

Currently the base class of SimpleMesage, "AbstractSomethingOrAnother" has a "getFullContent" method which creates a Sequenced Stream, using getHeader & getBody.

This is unnecessary in SimpleMessage, as the content stream is available in the private variables.

----------

Index: src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMessage.java
===================================================================
— src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMessage.java (revision 1345622)
+++ src/main/java/org/apache/james/mailbox/store/mail/model/impl/SimpleMessage.java (working copy)
@@ -96,6 +96,10 @@
}
}

+ @Override
+ public InputStream getFullContent() throws IOException

{ + return content.newStream(0, -1); + }

public Date getInternalDate() {
return internalDate;