From 817da386e456be422861ac6e974838e6eb6db836 Mon Sep 17 00:00:00 2001
From: Paul Phillips <paulp@improving.org>
Date: Fri, 2 Nov 2012 13:10:54 -0700
Subject: Fix for SI-6600, regression with ScalaNumber.

Not much in the end; I divided ScalaNumericConversions
into two traits such that the ScalaNumericAnyConversions can
be used in value classes, and ScalaNumericConversions can
override methods in ScalaNumber (since one trait cannot do
both those things.)

The fact that ScalaNumber is privileged for equality but a) extends
java.lang.Number and therefore b) cannot be a value class is something
we will want to revisit real soon.
---
 src/library/scala/math/ScalaNumericConversions.scala | 15 +++++++++++----
 src/library/scala/runtime/ScalaNumberProxy.scala     |  6 ++++--
 2 files changed, 15 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/library/scala/math/ScalaNumericConversions.scala b/src/library/scala/math/ScalaNumericConversions.scala
index edf243e5df..08c4118b00 100644
--- a/src/library/scala/math/ScalaNumericConversions.scala
+++ b/src/library/scala/math/ScalaNumericConversions.scala
@@ -10,15 +10,22 @@ package scala.math
 
 import java.{ lang => jl }
 
+/** A slightly more specific conversion trait for classes which
+ *  extend ScalaNumber (which excludes value classes.)
+ */
+trait ScalaNumericConversions extends ScalaNumber with ScalaNumericAnyConversions {
+  def underlying(): Object
+}
+
 /** Conversions which present a consistent conversion interface
- *  across all the numeric types.
+ *  across all the numeric types, suitable for use in value classes.
  */
-trait ScalaNumericConversions extends Any {
+trait ScalaNumericAnyConversions extends Any {
   def isWhole(): Boolean
   def underlying(): Any
 
-  def byteValue(): Byte = intValue().toByte
-  def shortValue(): Short = intValue().toShort
+  def byteValue(): Byte
+  def shortValue(): Short
   def intValue(): Int
   def longValue(): Long
   def floatValue(): Float
diff --git a/src/library/scala/runtime/ScalaNumberProxy.scala b/src/library/scala/runtime/ScalaNumberProxy.scala
index df2d209e3e..e461783423 100644
--- a/src/library/scala/runtime/ScalaNumberProxy.scala
+++ b/src/library/scala/runtime/ScalaNumberProxy.scala
@@ -9,7 +9,7 @@
 package scala.runtime
 
 import scala.collection.{ mutable, immutable }
-import scala.math.ScalaNumericConversions
+import scala.math.{ ScalaNumericConversions, ScalaNumericAnyConversions }
 import immutable.NumericRange
 import Proxy.Typed
 
@@ -20,7 +20,7 @@ import Proxy.Typed
  *  @version 2.9
  *  @since   2.9
  */
-trait ScalaNumberProxy[T] extends Any with ScalaNumericConversions with Typed[T] with OrderedProxy[T] {
+trait ScalaNumberProxy[T] extends Any with ScalaNumericAnyConversions with Typed[T] with OrderedProxy[T] {
   protected implicit def num: Numeric[T]
 
   def underlying()  = self.asInstanceOf[AnyRef]
@@ -28,6 +28,8 @@ trait ScalaNumberProxy[T] extends Any with ScalaNumericConversions with Typed[T]
   def floatValue()  = num.toFloat(self)
   def longValue()   = num.toLong(self)
   def intValue()    = num.toInt(self)
+  def byteValue()   = intValue.toByte
+  def shortValue()  = intValue.toShort
 
   def min(that: T): T = num.min(self, that)
   def max(that: T): T = num.max(self, that)
-- 
cgit v1.2.3