Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: Scala 2.11.0-M7
    • Fix Version/s: Scala 2.11.0-M8
    • Component/s: Collections
    • Labels:
      None

      Description

      Ready for this? This is a good one.

      scala> Vector() ++ Iterator(1)
      res9: scala.collection.immutable.Vector[Int] = Vector(1)
      
      scala> Vector() ++ Iterator(1, 2)
      res10: scala.collection.immutable.Vector[Int] = Vector(1, 2)
      
      scala> Vector() ++ Iterator(1, 2, 3)
      res11: scala.collection.immutable.Vector[Int] = Vector()
      

      was fine in 2.11.0-M5.

      I can't seem to find another combination of types besides Vector + Iterator on which it fails

        Activity

        Hide
        Seth Tisue added a comment - - edited

        OlegYch and datura on #scala figured out the culprit appears to be https://github.com/scala/scala/commit/4234b34dd4e6563ba8a9b5080cc6a0da021848d3#diff-59f3462485b74027de4fd5e9febcc81b from the fix for SI-7725. case _ => super.++(that) should be case _ => super.++(again)

        Show
        Seth Tisue added a comment - - edited OlegYch and datura on #scala figured out the culprit appears to be https://github.com/scala/scala/commit/4234b34dd4e6563ba8a9b5080cc6a0da021848d3#diff-59f3462485b74027de4fd5e9febcc81b from the fix for SI-7725 . case _ => super.++(that) should be case _ => super.++(again)
        Hide
        Rex Kerr added a comment -

        Whoops. Good catch. I'll fix it as soon as I have a moment.

        Show
        Rex Kerr added a comment - Whoops. Good catch. I'll fix it as soon as I have a moment.
        Show
        Adriaan Moors added a comment - https://github.com/scala/scala/pull/3213

          People

          • Assignee:
            Rex Kerr
            Reporter:
            Seth Tisue
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development