kosaki	2011-07-12 20:04:58 +0900 (Tue, 12 Jul 2011)

  New Revision: 32528

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=32528

  Log:
    * signal.c (reserved_signal_p): reverted a part of r32523.
      chikanaga noticed trap(:CHLD) has some realworld usecase.
    * test/ruby/test_signal.rb (TestSignal#test_reserved_signal):
      ditto.

  Modified files:
    trunk/ChangeLog
    trunk/signal.c
    trunk/test/ruby/test_signal.rb

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 32527)
+++ ChangeLog	(revision 32528)
@@ -1,3 +1,10 @@
+Tue Jul 12 20:02:35 2011  KOSAKI Motohiro  <kosaki.motohiro@g...>
+
+	* signal.c (reserved_signal_p): reverted a part of r32523.
+	  chikanaga noticed trap(:CHLD) has some realworld usecase.
+	* test/ruby/test_signal.rb (TestSignal#test_reserved_signal):
+	  ditto.
+
 Tue Jul 12 17:12:45 2011  Yukihiro Matsumoto  <matz@r...>
 
 	* vm_method.c (rb_add_method): should not call method_added hook
Index: test/ruby/test_signal.rb
===================================================================
--- test/ruby/test_signal.rb	(revision 32527)
+++ test/ruby/test_signal.rb	(revision 32528)
@@ -237,12 +237,6 @@
     assert_raise(ArgumentError) {
       Signal.trap(:VTALRM) {}
     }
-    assert_raise(ArgumentError) {
-      Signal.trap(:CHLD) {}
-    }
-    assert_raise(ArgumentError) {
-      Signal.trap(:CLD) {}
-    }
   end
 
 end
Index: signal.c
===================================================================
--- signal.c	(revision 32527)
+++ signal.c	(revision 32528)
@@ -896,16 +896,6 @@
 	return 1;
 #endif
 
-/* On some OSs, wait() never return if SIGCHLD handler is installed. */
-#ifdef SIGCHLD
-    if (signo == SIGCHLD)
-	return 1;
-#endif
-#ifdef SIGCLD
-    if (signo == SIGCLD)
-	return 1;
-#endif
-
     return 0;
 }
 

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/