Project

General

Profile

Bug #6117

Transfers allowed into Fibers that have resumed into other fibers

Added by loganb (Logan Bowers) about 5 years ago. Updated 4 months ago.

Status:
Feedback
Priority:
Normal
Target version:
-
ruby -v:
ruby 2.0.0dev (2012-03-06) [x86_64-darwin11.3.0]
Backport:
[ruby-core:43088]

Description

Consider the following code:

  require 'fiber'

  def foo
    order = []
    main_fiber = Fiber.current
    a = Fiber.new do 
      order << 2
      x = Fiber.new do
        order << 4
        main_fiber.transfer
        order << 6
      end
      order << 3
      x.resume
      order << 7
    end

    order << 1
    a.transfer
    order << 5
    a.transfer
    order << 8
  end

It produces output:

  [1, 2, 3, 4, 5, 7, 8]

Note: the missing 6. I'd expect either the array to be 1-8 OR for an exception to be raised after 5 when the main fiber attempts to transfer into a again.

Either it shouldn't be possible to transfer into a Fiber that is blocked resuming another Fiber, OR it should automatically resume the sub-Fiber.

History

#1 [ruby-core:43225] Updated by mrkn (Kenta Murata) about 5 years ago

  • Assignee set to ko1 (Koichi Sasada)

#2 Updated by shyouhei (Shyouhei Urabe) about 5 years ago

  • Status changed from Open to Assigned

#3 [ruby-core:48573] Updated by ko1 (Koichi Sasada) over 4 years ago

  • Target version set to 2.0.0

Please give me a time.

#4 [ruby-core:50133] Updated by ko1 (Koichi Sasada) over 4 years ago

Exception seems good.
transfer' andresume' should not be mix.

Any volunteer?
or it will be tagged `next minor'.

#5 [ruby-core:50994] Updated by ko1 (Koichi Sasada) over 4 years ago

Any volunteers?

#6 [ruby-core:51276] Updated by ko1 (Koichi Sasada) over 4 years ago

  • Target version changed from 2.0.0 to next minor

no volunteer.

#7 Updated by ko1 (Koichi Sasada) almost 2 years ago

  • Status changed from Assigned to Feedback
  • Description updated (diff)

#8 [ruby-core:79354] Updated by ko1 (Koichi Sasada) 4 months ago

Can we change this spec yet?

Also available in: Atom PDF