Skip to content

Fix Fiber#alive? regression.#1949

Closed
havenwood wants to merge 1 commit intojruby:jruby-1_7from
havenwood:jruby-1_7
Closed

Fix Fiber#alive? regression.#1949
havenwood wants to merge 1 commit intojruby:jruby-1_7from
havenwood:jruby-1_7

Conversation

@havenwood
Copy link

In JRuby 1.7.10:

require 'fiber'

fiber = Fiber.new {}

fiber.alive? #=> true
fiber.resume
fiber.alive? #=> false

In JRuby 1.7.11 through 1.7.15:

require 'fiber'

fiber = Fiber.new {}

fiber.alive? #=> true
fiber.resume
fiber.alive? #=> true

It looks like this is already fixed on master.

Closes #1789.

In JRuby 1.7.10:
```ruby
require 'fiber'

fiber = Fiber.new {}

fiber.alive? #=> true
fiber.resume
fiber.alive? #=> false
```

In JRuby 1.7.11 through 1.7.15:
```ruby
require 'fiber'

fiber = Fiber.new {}

fiber.alive? #=> true
fiber.resume
fiber.alive? #=> true
```

It looks like this is [already fixed on master](https://github.com/jruby/jruby/blob/master/core/src/main/java/org/jruby/ext/fiber/ThreadFiber.java#L224).
@havenwood
Copy link
Author

Closing since this is fixed in JRuby 1.7.16. Sorry for the noise!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants