Scala Programming Language
  1. Scala Programming Language
  2. SI-3628

NPE typechecking invalid code when continuations are enabled

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Scala 2.10.0
    • Component/s: Misc Compiler
    • Labels:
      None
    • Environment:

      continuations

      Description

      [Elided very sloppy report, replacing with substance.]

      import scala.actors.Actor._
      
      object Test {
        val impl: Actor = actor {
          loop {
            react { 
              case 1 => impl ! 2
            }
          }
        }
      }
      

      Notice scala.actors.Actor is not imported. scalac crashes during compilation, but only if continuations are enabled.

        Activity

        Hide
        Philipp Haller added a comment -

        (In r22539) Added test case for see SI-3628 in pending.

        Show
        Philipp Haller added a comment - (In r22539) Added test case for see SI-3628 in pending.
        Hide
        Topher added a comment -

        This has already been fixed. I have attached a patch that moves phaller's test from pending into files/continuations-neg and corrects the expected output. It's not a very deep patch, but it does allow this bug to be closed.

        Being that this moves files, you'll want to apply it with:

        patch -p0 -E -i t3628.diff
        svn rm test/pending/continuations-neg/t3628.check test/pending/continuations-neg/t3628.scala
        svn add test/files/continuations-neg/t3628.check test/files/continuations-neg/t3628.scala
        
        Show
        Topher added a comment - This has already been fixed. I have attached a patch that moves phaller's test from pending into files/continuations-neg and corrects the expected output. It's not a very deep patch, but it does allow this bug to be closed. Being that this moves files, you'll want to apply it with: patch -p0 -E -i t3628.diff svn rm test/pending/continuations-neg/t3628.check test/pending/continuations-neg/t3628.scala svn add test/files/continuations-neg/t3628.check test/files/continuations-neg/t3628.scala
        Hide
        Commit Message Bot added a comment -

        (extempore in r25418) Moved test out of pending, closes SI-3628, no review.

        Show
        Commit Message Bot added a comment - (extempore in r25418 ) Moved test out of pending, closes SI-3628 , no review.

          People

          • Assignee:
            Tiark Rompf
            Reporter:
            Ittay Dror
            TracCC:
            Paul Phillips
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development