From 5b83dc366ed1614e56ad4a65f61c9775fe698b2d Mon Sep 17 00:00:00 2001 From: Oliver Schulz Date: Thu, 31 May 2012 00:44:21 +0200 Subject: [PATCH] Removed unnecessary implementations of ByteString.clone * As all ByteString implementations are immutable, an explicit override of Object.clone doesn't seem necessary. --- akka-actor/src/main/scala/akka/util/ByteString.scala | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/akka-actor/src/main/scala/akka/util/ByteString.scala b/akka-actor/src/main/scala/akka/util/ByteString.scala index 8aeed6daf2..98623429e3 100644 --- a/akka-actor/src/main/scala/akka/util/ByteString.scala +++ b/akka-actor/src/main/scala/akka/util/ByteString.scala @@ -79,8 +79,6 @@ object ByteString { private[akka] def toByteString1: ByteString1 = ByteString1(bytes) - override def clone: ByteString1C = new ByteString1C(toArray) - def asByteBuffer: ByteBuffer = toByteString1.asByteBuffer @@ -121,12 +119,10 @@ object ByteString { throw new IndexOutOfBoundsException(index.toString) } - override def clone: CompactByteString = ByteString1C(toArray) - def isCompact: Boolean = (length == bytes.length) def compact: CompactByteString = - if (isCompact) ByteString1C(bytes) else clone + if (isCompact) ByteString1C(bytes) else ByteString1C(toArray) def asByteBuffer: ByteBuffer = { val buffer = ByteBuffer.wrap(bytes, startIndex, length).asReadOnlyBuffer