diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | prelude.rb | 16 | ||||
-rw-r--r-- | thread.c | 18 |
3 files changed, 23 insertions, 16 deletions
@@ -1,3 +1,8 @@ +Tue Nov 20 09:20:49 2012 KOSAKI Motohiro <kosaki.motohiro@gmail.com> + + * prelude.rb: Moved Mutex#synchronize to + * thread.c (rb_mutex_synchronize_m): here. [Bug #4266] + Tue Nov 20 08:36:15 2012 KOSAKI Motohiro <kosaki.motohiro@gmail.com> * signal.c (sig_signame): implements Signal.signame method diff --git a/prelude.rb b/prelude.rb index 4b6ab1a677..2b371e7134 100644 --- a/prelude.rb +++ b/prelude.rb @@ -1,19 +1,3 @@ -class Mutex - # call-seq: - # mutex.synchronize { ... } - # - # Obtains a lock, runs the block, and releases the lock when the - # block completes. See the example under Mutex. - def synchronize - self.lock - begin - yield - ensure - self.unlock rescue nil - end - end -end - class Thread MUTEX_FOR_THREAD_EXCLUSIVE = Mutex.new # :nodoc: @@ -4286,6 +4286,23 @@ rb_mutex_synchronize(VALUE mutex, VALUE (*func)(VALUE arg), VALUE arg) } /* + * call-seq: + * mutex.synchronize { ... } -> result of the block + * + * Obtains a lock, runs the block, and releases the lock when the block + * completes. See the example under +Mutex+. + */ +static VALUE +rb_mutex_synchronize_m(VALUE self, VALUE args) +{ + if (!rb_block_given_p()) { + rb_raise(rb_eThreadError, "must be called with a block"); + } + + return rb_mutex_synchronize(self, rb_yield, Qnil); +} + +/* * Document-class: ThreadShield */ static void @@ -4740,6 +4757,7 @@ Init_Thread(void) rb_define_method(rb_cMutex, "lock", rb_mutex_lock, 0); rb_define_method(rb_cMutex, "unlock", rb_mutex_unlock, 0); rb_define_method(rb_cMutex, "sleep", mutex_sleep, -1); + rb_define_method(rb_cMutex, "synchronize", rb_mutex_synchronize_m, 0); recursive_key = rb_intern("__recursive_key__"); rb_eThreadError = rb_define_class("ThreadError", rb_eStandardError); |