Skip to content

Commit dc6680b

Browse files
mbezoyanjenkins
authored andcommitted
[finagle-core] Don't make an unnecessary copy of the array with frames in LengthFieldFramer
Problem: We unnecessarily copy the array with extracted frames into an indexed seq. Solution: Don't copy Differential Revision: https://phabricator.twitter.biz/D1164516
1 parent be778a3 commit dc6680b

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

finagle-core/src/main/scala/com/twitter/finagle/decoder/LengthFieldFramer.scala

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.twitter.finagle.decoder
22

3-
import com.twitter.io.{Buf, ByteReader}
3+
import com.twitter.io.Buf
4+
import com.twitter.io.ByteReader
45
import scala.collection.mutable.ArrayBuffer
56

67
private[twitter] object LengthFieldFramer {
@@ -150,7 +151,7 @@ private[twitter] class LengthFieldFramer(
150151
}
151152

152153
accum = accum.slice(frameCursor, accum.length)
153-
frames.toIndexedSeq
154+
frames.asInstanceOf[IndexedSeq[Buf]]
154155
} else {
155156
NoFrames
156157
}

0 commit comments

Comments
 (0)