diff options
author | Adriaan Moors <adriaan.moors@typesafe.com> | 2015-02-13 11:25:25 -0800 |
---|---|---|
committer | Adriaan Moors <adriaan.moors@typesafe.com> | 2015-02-13 11:25:25 -0800 |
commit | 788f38871ca52bc22967dca22355f6a1affb4e06 (patch) | |
tree | c538cd6e4f548d9bad6b9533a521951e284f455c /src/library | |
parent | f7bc59bc2f6815cf6ca9cc17173f4335d2a0fd0b (diff) | |
download | scala-788f38871ca52bc22967dca22355f6a1affb4e06.tar.gz scala-788f38871ca52bc22967dca22355f6a1affb4e06.tar.bz2 scala-788f38871ca52bc22967dca22355f6a1affb4e06.zip |
Simplify `ClassTag.unapply`
Inline overloaded calls to `unapply`, so we can get rid of them in 2.13.
Note that theres a lot of `box(unbox(x))` going on behind the scenes.
Is this needed?
Diffstat (limited to 'src/library')
-rw-r--r-- | src/library/scala/reflect/ClassTag.scala | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/src/library/scala/reflect/ClassTag.scala b/src/library/scala/reflect/ClassTag.scala index e83f689ca7..b9b4772870 100644 --- a/src/library/scala/reflect/ClassTag.scala +++ b/src/library/scala/reflect/ClassTag.scala @@ -71,20 +71,21 @@ trait ClassTag[T] extends ClassManifestDeprecatedApis[T] with Equals with Serial */ def unapply(x: Any): Option[T] = x match { case null => None - case b: Byte => unapply(b) - case s: Short => unapply(s) - case c: Char => unapply(c) - case i: Int => unapply(i) - case l: Long => unapply(l) - case f: Float => unapply(f) - case d: Double => unapply(d) - case b: Boolean => unapply(b) - case u: Unit => unapply(u) - case a: Any => unapplyImpl(a) + case x: Byte => unapplyImpl(x, classOf[Byte]) // erases to: if (x instanceof Byte) unapplyImpl(BoxesRunTime.boxToByte(BoxesRunTime.unboxToByte(x)), Byte.TYPE) + case x: Short => unapplyImpl(x, classOf[Short]) + case x: Char => unapplyImpl(x, classOf[Char]) + case x: Int => unapplyImpl(x, classOf[Int]) + case x: Long => unapplyImpl(x, classOf[Long]) + case x: Float => unapplyImpl(x, classOf[Float]) + case x: Double => unapplyImpl(x, classOf[Double]) + case x: Boolean => unapplyImpl(x, classOf[Boolean]) + case x: Unit => unapplyImpl(x, classOf[Unit]) + // TODO: move this next case up and remove the redundant check in unapplyImpl? + case _ if runtimeClass isInstance x => Some(x.asInstanceOf[T]) + case _ => None } - // TODO: Inline the bodies of these into the Any-accepting unapply overload above and delete them. - // This cannot be done until at least 2.12.0 for reasons of binary compatibility + // TODO: deprecate overloads in 2.12.0, remove in 2.13.0 def unapply(x: Byte) : Option[T] = unapplyImpl(x, classOf[Byte]) def unapply(x: Short) : Option[T] = unapplyImpl(x, classOf[Short]) def unapply(x: Char) : Option[T] = unapplyImpl(x, classOf[Char]) @@ -95,10 +96,9 @@ trait ClassTag[T] extends ClassManifestDeprecatedApis[T] with Equals with Serial def unapply(x: Boolean) : Option[T] = unapplyImpl(x, classOf[Boolean]) def unapply(x: Unit) : Option[T] = unapplyImpl(x, classOf[Unit]) - private[this] def unapplyImpl(x: Any, alternative: jClass[_] = null): Option[T] = { - val conforms = runtimeClass.isInstance(x) || (alternative != null && runtimeClass.isAssignableFrom(alternative)) - if (conforms) Some(x.asInstanceOf[T]) else None - } + private[this] def unapplyImpl(x: Any, primitiveCls: java.lang.Class[_]): Option[T] = + if (runtimeClass.isInstance(x) || runtimeClass.isAssignableFrom(primitiveCls)) Some(x.asInstanceOf[T]) + else None // case class accessories override def canEqual(x: Any) = x.isInstanceOf[ClassTag[_]] |