Skip to content

Commit

Permalink
* signal.c (rb_get_next_signal): removed pointless signal
Browse files Browse the repository at this point in the history
  disabling. pthread_sigmask() only changes current thread
  mask.



git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37727 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
kosaki committed Nov 19, 2012
1 parent 562cb0a commit 6d9f767
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
6 changes: 6 additions & 0 deletions ChangeLog
@@ -1,3 +1,9 @@
Tue Nov 20 09:48:34 2012 KOSAKI Motohiro <kosaki.motohiro@gmail.com>

* 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@gmail.com>

* signal.c (install_sighandler): added comments why we need
Expand Down
8 changes: 2 additions & 6 deletions signal.c
Expand Up @@ -562,12 +562,8 @@ rb_get_next_signal(void)
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;
}
Expand Down

0 comments on commit 6d9f767

Please sign in to comment.