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

elecharny pushed a commit to branch mina-write-request
in repository https://gitbox.apache.org/repos/asf/mina.git


The following commit(s) were added to refs/heads/mina-write-request by this 
push:
     new 231b044  Inverted the duplicate, as suggested by Jonathan.
231b044 is described below

commit 231b044d0ea1f968f7a07dec7d2428fbd3b38667
Author: emmanuel lecharny <elecha...@apache.org>
AuthorDate: Tue Apr 2 16:31:06 2019 +0200

    Inverted the duplicate, as suggested by Jonathan.
---
 .../main/java/org/apache/mina/core/write/DefaultWriteRequest.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/mina-core/src/main/java/org/apache/mina/core/write/DefaultWriteRequest.java 
b/mina-core/src/main/java/org/apache/mina/core/write/DefaultWriteRequest.java
index 91c1330..8324c6a 100644
--- 
a/mina-core/src/main/java/org/apache/mina/core/write/DefaultWriteRequest.java
+++ 
b/mina-core/src/main/java/org/apache/mina/core/write/DefaultWriteRequest.java
@@ -221,14 +221,14 @@ public class DefaultWriteRequest implements WriteRequest {
         }
 
         this.message = message;
+        originalMessage = message;
         
         if (message instanceof IoBuffer) {
             // duplicate it, so that any modification made on it
             // won't change the original message
-            this.originalMessage = ((IoBuffer)message).duplicate();
-        } else {
-            originalMessage = message;
+            this.message = ((IoBuffer)message).duplicate();
         }
+
         
         this.future = future;
         this.destination = destination;

Reply via email to