I wrote a warning when nullary methods return U...

I wrote a warning when nullary methods return Unit. I wimped out of
including it in this patch because we had about 200 of them, and that's
what is fixed in this patch. I will add the warning to some kind of
"-Xlint" feature after 2.9.

This is motivated at least partly by the resolution of #4506, which
indicates the distinction between "def foo()" and "def foo" will
continue to jab its pointy stick into our eyes, so I believe we have a
minimal duty of at least following our own advice about what they mean
and not making a semirandom choice as to whether a method has parens or
not. Review by community.

2 files changed, 4 insertions, 4 deletions

diff --git a/test/files/continuations-neg/lazy.check b/test/files/continuations-neg/lazy.check
index bfa44c59a4..b8c6887409 100644
--- a/test/files/continuations-neg/lazy.check
+++ b/test/files/continuations-neg/lazy.check
@@ -1,6 +1,6 @@
 lazy.scala:5: error: type mismatch;
  found   : Unit @scala.util.continuations.cpsParam[Unit,Unit]
  required: Unit
-  def foo = {
-            ^
+  def foo() = {
+              ^
 one error found
diff --git a/test/files/continuations-neg/lazy.scala b/test/files/continuations-neg/lazy.scala
index dffc57ffa0..3c8169dd46 100644
--- a/test/files/continuations-neg/lazy.scala
+++ b/test/files/continuations-neg/lazy.scala
@@ -2,14 +2,14 @@ import scala.util.continuations._
 
 object Test {
 
-  def foo = {
+  def foo() = {
     lazy val x = shift((k:Unit=>Unit)=>k())
     println(x)
   }
 
   def main(args: Array[String]) {
     reset {
-      foo
+      foo()
     }
   }