[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]