diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-12-14 03:20:02 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-12-14 03:20:02 +0000 |
commit | 50c1985555b00bf8e19646a5aad7e881ac84401b (patch) | |
tree | 4d211b196572fc84e12c2c693b0fb474b7dfa2a2 /load.c | |
parent | 23f9e7460462efa9273c05b8c616ca9dfa6e24af (diff) |
* load.c (load_unlock): release loading barrier and then remove it
from loading_table if it is not in-use. [Bug #5754]
* thread.c (rb_barrier_release, rb_barrier_destroy): return
whether any other threads are waiting on it.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34039 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'load.c')
-rw-r--r-- | load.c | 10 |
1 files changed, 4 insertions, 6 deletions
@@ -427,12 +427,10 @@ load_unlock(const char *ftptr, int done) if (!st_lookup(loading_tbl, key, &data)) return; barrier = (VALUE)data; - if (rb_barrier_waiting(barrier) || - (st_delete(loading_tbl, &key, &data) && (xfree((char *)key), 1))) { - if (done) - rb_barrier_destroy(barrier); - else - rb_barrier_release(barrier); + if (!(done ? rb_barrier_destroy(barrier) : rb_barrier_release(barrier))) { + if (st_delete(loading_tbl, &key, &data)) { + xfree((char *)key); + } } } } |