ruby-changes:25670
From: kosaki <ko1@a...>
Date: Mon, 19 Nov 2012 19:57:49 +0900 (JST)
Subject: [ruby-changes:25670] kosaki:r37727 (trunk): * signal.c (rb_get_next_signal): removed pointless signal
kosaki 2012-11-19 19:54:55 +0900 (Mon, 19 Nov 2012) New Revision: 37727 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=37727 Log: * signal.c (rb_get_next_signal): removed pointless signal disabling. pthread_sigmask() only changes current thread mask. Modified files: trunk/ChangeLog trunk/signal.c Index: ChangeLog =================================================================== --- ChangeLog (revision 37726) +++ ChangeLog (revision 37727) @@ -1,3 +1,9 @@ +Tue Nov 20 09:48:34 2012 KOSAKI Motohiro <kosaki.motohiro@g...> + + * signal.c (rb_get_next_signal): removed pointless signal + disabling. pthread_sigmask() only changes current thread + mask. + Tue Nov 20 09:36:55 2012 KOSAKI Motohiro <kosaki.motohiro@g...> * signal.c (install_sighandler): added comments why we need Index: signal.c =================================================================== --- signal.c (revision 37726) +++ signal.c (revision 37727) @@ -562,12 +562,8 @@ if (signal_buff.size != 0) { for (i=1; i<RUBY_NSIG; i++) { if (signal_buff.cnt[i] > 0) { - rb_disable_interrupt(); - { - ATOMIC_DEC(signal_buff.cnt[i]); - ATOMIC_DEC(signal_buff.size); - } - rb_enable_interrupt(); + ATOMIC_DEC(signal_buff.cnt[i]); + ATOMIC_DEC(signal_buff.size); sig = i; break; } -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/