diff options
| author | Sébastien Doeraene <sjrdoeraene@gmail.com> | 2016-04-13 14:55:02 +0200 |
|---|---|---|
| committer | Sébastien Doeraene <sjrdoeraene@gmail.com> | 2016-04-13 16:20:46 +0200 |
| commit | 8fc0516d6deeb84af9bac5370f1d21511e0f86d6 (patch) | |
| tree | 74ee8fdc85c8c7fc8e8e21b954a34e8781044475 /src/library/scala/collection/mutable/WrappedArrayBuilder.scala | |
| parent | 51996c8474a06656f8c21302af2b4151bdc2393e (diff) | |
| download | scala-8fc0516d6deeb84af9bac5370f1d21511e0f86d6.tar.gz scala-8fc0516d6deeb84af9bac5370f1d21511e0f86d6.tar.bz2 scala-8fc0516d6deeb84af9bac5370f1d21511e0f86d6.zip | |
Inline ScalaRunTime.arrayElementClass at call sites.
This method was awful. Not only it was using run-time type
tests to essentially encode compile-time overloading. But
it also did 2 slightly different things for the Class case
and ClassTag case.
All in all, it is much more readable to inline the
appropriate implementation at every call site.
Diffstat (limited to 'src/library/scala/collection/mutable/WrappedArrayBuilder.scala')
| -rw-r--r-- | src/library/scala/collection/mutable/WrappedArrayBuilder.scala | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/library/scala/collection/mutable/WrappedArrayBuilder.scala b/src/library/scala/collection/mutable/WrappedArrayBuilder.scala index c4781321d7..5bc5811450 100644 --- a/src/library/scala/collection/mutable/WrappedArrayBuilder.scala +++ b/src/library/scala/collection/mutable/WrappedArrayBuilder.scala @@ -13,7 +13,6 @@ package collection package mutable import scala.reflect.ClassTag -import scala.runtime.ScalaRunTime._ /** A builder class for arrays. * @@ -34,7 +33,7 @@ class WrappedArrayBuilder[A](tag: ClassTag[A]) extends ReusableBuilder[A, Wrappe private var size: Int = 0 private def mkArray(size: Int): WrappedArray[A] = { - val runtimeClass = arrayElementClass(tag) + val runtimeClass = tag.runtimeClass val newelems = runtimeClass match { case java.lang.Byte.TYPE => new WrappedArray.ofByte(new Array[Byte](size)).asInstanceOf[WrappedArray[A]] case java.lang.Short.TYPE => new WrappedArray.ofShort(new Array[Short](size)).asInstanceOf[WrappedArray[A]] |