Project

General

Profile

Feature #13639

Add "RTMIN" and "RTMAX" to Signal.list

Added by ksss (Yuki Kurihara) over 2 years ago. Updated about 2 years ago.

Status:
Open
Priority:
Normal
Assignee:
-
Target version:
-
[ruby-core:81617]

Description

I propose to add the key "RTMIN" and "RTMAX" to Signal.list

Currently, We can't know signal number for realtime-signal at own environment in Ruby.
And it's not potable if hard code number literal.

So, How about this idea?

Use case:

unless Signal.list["RTMIN"]
  raise "realtime-signal does not supported"
end

MY_SIGNAL = Signal.list["RTMIN"] + 1

Signal.trap(MY_SIGNAL) do |i|
  puts "Hello MY_SIGNAL"
end

Process.kill(MY_SIGNAL, $$)
#=> "Hello MY_SIGNAL"

Patch:

diff --git a/signal.c b/signal.c
index 8ee0963b8a..f17836350a 100644
--- a/signal.c
+++ b/signal.c
@@ -1381,6 +1381,12 @@ sig_list(void)
     for (sigs = siglist; sigs->signm; sigs++) {
        rb_hash_aset(h, rb_fstring_cstr(sigs->signm), INT2FIX(sigs->signo));
     }
+#ifdef SIGRTMIN
+    rb_hash_aset(h, rb_fstring_cstr("RTMIN"), INT2FIX(SIGRTMIN));
+#endif
+#ifdef SIGRTMAX
+    rb_hash_aset(h, rb_fstring_cstr("RTMAX"), INT2FIX(SIGRTMAX));
+#endif
     return h;
 }

Inspired by https://github.com/ksss/mruby-signal/pull/3 and @takumakume

History

Updated by matz (Yukihiro Matsumoto) about 2 years ago

I propose

  • Signal::RTMIN
  • Signal::RTMAX

instead.

Matz.

Also available in: Atom PDF