Skip to content

Commit

Permalink
Revert "Fix deadlock when tearing down runtime during debugging"
Browse files Browse the repository at this point in the history
This reverts commit f9d704c.

New issue #4419 opened to address this revert.
  • Loading branch information
enebo committed Jan 3, 2017
1 parent 0bfe15c commit 57ba4fd
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions core/src/main/java/org/jruby/Ruby.java
Expand Up @@ -3167,10 +3167,6 @@ public void tearDown(boolean systemExit) {

final ThreadContext context = getCurrentContext();

// Disable event hooks during runtime teardown
// This avoids deadlocks if some other thread is holding a debugger lock while we're trying to exit
context.setEventHooksEnabled(false);

// FIXME: 73df3d230b9d92c7237d581c6366df1b92ad9b2b exposed no toplevel scope existing anymore (I think the
// bogus scope I removed was playing surrogate toplevel scope and wallpapering this bug). For now, add a
// bogus scope back for at_exit block run. This is buggy if at_exit is capturing vars.
Expand Down

0 comments on commit 57ba4fd

Please sign in to comment.