diff options
author | Paul Phillips <paulp@improving.org> | 2011-07-29 15:38:50 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2011-07-29 15:38:50 +0000 |
commit | 4e488a60594664046c3449e1aa2239adca7a012e (patch) | |
tree | e02baffa12f57600c28e0109adf66e4ac0a00920 /src/actors/scala/actors/Future.scala | |
parent | cda84e7f2145673e4e14cb886598fe19cd48585a (diff) | |
download | scala-4e488a60594664046c3449e1aa2239adca7a012e.tar.gz scala-4e488a60594664046c3449e1aa2239adca7a012e.tar.bz2 scala-4e488a60594664046c3449e1aa2239adca7a012e.zip |
Added two new compiler options:
-Ywarn-adapted-args // also included in -Xlint
-Yno-adapted-args
The former warns when a () is inserted or an argument list is implicitly
tupled. The latter errors under the same conditions. Using these options
I found several bugs in the distribution which would otherwise be nearly
impossible to spot. These bugs were innocuous (I think) but similar bugs
could easily be (and have been) otherwise.
Certain particularly threatening scenarios are at minimum warned about
regardless of options given. Closes SI-4851, no review.
Diffstat (limited to 'src/actors/scala/actors/Future.scala')
-rw-r--r-- | src/actors/scala/actors/Future.scala | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/actors/scala/actors/Future.scala b/src/actors/scala/actors/Future.scala index c6b575d0ee..aaaf5691f1 100644 --- a/src/actors/scala/actors/Future.scala +++ b/src/actors/scala/actors/Future.scala @@ -102,7 +102,9 @@ private class FutureActor[T](fun: SyncVar[T] => Unit, channel: Channel[T]) exten loop { react { - case Eval => reply() + // This is calling ReplyReactor#reply(msg: Any). + // Was: reply(). Now: reply(()). + case Eval => reply(()) } } } |