summaryrefslogtreecommitdiff
path: root/docs
diff options
context:
space:
mode:
authorJason Zaugg <jzaugg@gmail.com>2015-04-24 15:41:31 +1000
committerJason Zaugg <jzaugg@gmail.com>2015-05-15 16:56:54 +1000
commit6ad9b44b27ede70ec723204bd80361d60f448c1a (patch)
tree223b11452c2d68470d6ae8768969098dd14502d1 /docs
parent99d3ab3be01ccd347d79162ed412aaf1ff0dff36 (diff)
downloadscala-6ad9b44b27ede70ec723204bd80361d60f448c1a.tar.gz
scala-6ad9b44b27ede70ec723204bd80361d60f448c1a.tar.bz2
scala-6ad9b44b27ede70ec723204bd80361d60f448c1a.zip
[indylambda] Relieve LambdaMetafactory of boxing duties
`LambdaMetafactory` generates code to perform a limited number of type adaptations when delegating from its implementation of the functional interface method to the lambda target method. These adaptations are: numeric widening, casting, boxing and unboxing. However, the semantics of unboxing numerics in Java differs to Scala: they treat `UNBOX(null)` as cause to raise a `NullPointerException`, Scala (in `BoxesRuntime.unboxTo{Byte,Short,...}`) reinterprets the null as zero. Furthermore, Java has no idea how to adapt between a value class and its wrapped type, nor from a void return to `BoxedUnit`. This commit detects when the lambda target method would require such adaptation. If it does, an extra method, `$anonfun$1$adapted` is created to perform the adaptation, and this is used as the target of the lambda. This obviates the use of `JProcedureN` for `Unit` returning lambdas, we know use `JFunctionN` as the functional interface and bind this to an `$adapted` method that summons the instance of `BoxedUnit` after calling the `void` returning lambda target. The enclosed test cases fail without boxing changes. They don't execute with indylambda enabled under regular partest runs yet, you need to add scala-java8-compat to scala-library and pass the SCALAC_OPTS to partest manually to try this out, as described in https://github.com/scala/scala/pull/4463. Once we enable indylambda by default, however, this test will exercise the code in this patch all the time. It is also possible to run the tests with: ``` % curl https://oss.sonatype.org/content/repositories/releases/org/scala-lang/modules/scala-java8-compat_2.11/0.4.0/scala-java8-compat_2.11-0.4.0.jar > scala-java8-compat_2.11-0.4.0.jar % export INDYLAMBDA="-Ydelambdafy:method -Ybackend:GenBCode -target:jvm-1.8 -classpath .:scala-java8-compat_2.11-0.4.0.jar" qscalac $INDYLAMBDA test/files/run/indylambda-boxing/*.scala && qscala $INDYLAMBDA Test ```
Diffstat (limited to 'docs')
0 files changed, 0 insertions, 0 deletions