Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: Scala 2.10.2-RC1, Scala 2.11.0-M3
    • Component/s: None
    • Labels:
      None
    • Environment:

      Scala version 2.10.2-20130515-155912-70e2ead142

      Description

      With 2.10.2-SNAPSHOT (2.10.2-20130515-155912-70e2ead142) compilation error is reported for the following code in akka-testkit.
      https://github.com/akka/akka/blob/master/akka-testkit/src/test/scala/akka/testkit/AkkaSpecSpec.scala#L67

              implicit val davyJones = otherSystem.actorOf(Props(new Actor {
                def receive = {
                  case m: DeadLetter ⇒ locker :+= m
                  case "Die!"        ⇒ sender ! "finally gone"; context.stop(self)
                }
              }), "davyJones")
      
      [error] /Users/patrik/dev/akka/akka-testkit/src/test/scala/akka/testkit/AkkaSpecSpec.scala:68: recursive value davyJones needs type
      [error]           def receive = {
      [error]               ^
      

      It compiles fine when I remove the implicit, which we actually don't need here.

      This is compiling fine with 2.10.1.

        Issue Links

          Activity

          Hide
          Jason Zaugg added a comment -

          I vote to revert for 2.10.2, but leave the change in master. We can poke and test things a bit more in master to understand the scope of the change (which does seem pretty small.) I suspect that if we had an INFERRED flag on symbols we could make this deterministic.

          Show
          Jason Zaugg added a comment - I vote to revert for 2.10.2, but leave the change in master. We can poke and test things a bit more in master to understand the scope of the change (which does seem pretty small.) I suspect that if we had an INFERRED flag on symbols we could make this deterministic.
          Hide
          James Iry added a comment -

          Paul, do you want to own the revert?

          Show
          James Iry added a comment - Paul, do you want to own the revert?
          Hide
          Paul Phillips added a comment -

          I can revert it as long as nobody is going to insist on a test. The difficulty of testing it is the thing which makes it "ridiculous" so my hands are tied. If jason can't extract a test from scalatest, I guarantee I can't.

          Show
          Paul Phillips added a comment - I can revert it as long as nobody is going to insist on a test. The difficulty of testing it is the thing which makes it "ridiculous" so my hands are tied. If jason can't extract a test from scalatest, I guarantee I can't.
          Hide
          Adriaan Moors added a comment -

          A l'impossible nul n'est tenu. It's the law. (In some places.)

          Show
          Adriaan Moors added a comment - A l'impossible nul n'est tenu. It's the law. (In some places.)
          Show
          Jason Zaugg added a comment - https://github.com/scala/scala/pull/2590

            People

            • Assignee:
              Paul Phillips
              Reporter:
              Patrik Nordwall
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development