You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by el...@apache.org on 2019/04/02 14:31:11 UTC

[mina] branch mina-write-request updated: Inverted the duplicate, as suggested by Jonathan.

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 <el...@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;