[ruby-dev:8397] Re: [PATCH] ruby -e system

小松です。

In the message of [ruby-dev:8392] Re: [PATCH] ruby -e system
    on 1999/11/20 17:48:23 [e-mail] (Yukihiro Matsumoto) wrote:
||ちょっと気になるのは、
||    fflush(stdout);
||    fflush(stderr);
||がNT, __EMX__, DJGPPの枝に必要かという点と、
||DJGPPの場合、argv[0]が読めるので、現状、
||    ruby -e system
||でshellが立ち上がりますが、ArgumentErrorにしてしまっても
||良いかということです。
|
|前者は分かりません。後者はErrorで良いと思います。

失礼、前者の件は最初から確認しておくべきでした。

以下のようにfflush()はNTの枝には必要、DJGPPの枝は不要です。
__EMX__は分かりませんが、あっても害はないでしょう。

C:\lang\ruby>ruby -v
ruby 1.5.0 (1999-11-17) [i386-mswin32]

C:\lang\ruby>type x
def

C:\lang\ruby>type x.rb
STDERR.print "abc"
system "cat x"
STDERR.print "\n"

C:\lang\ruby>ruby x.rb > out 2>&1

C:\lang\ruby>type out
def
abc

C:\lang\ruby>rubydj -v
ruby 1.4.2 (1999-09-18) [i386-djgpp]

C:\lang\ruby>rubydj s.rb > out 2>&1

C:\lang\ruby>type out
abcdef


C:\lang\ruby>

--
小松克行 / Katsuyuki Komatsu <[e-mail]>

--- process.c.dist	Fri Oct 29 18:25:31 1999
+++ process.c	Mon Nov 22 12:05:10 1999
@@ -601,4 +601,11 @@ rb_f_system(argc, argv)
     int state;
 
+    fflush(stdout);
+    fflush(stderr);
+    if (argc == 0) {
+	rb_last_status = INT2FIX(0);
+	rb_raise(rb_eArgError, "wrong # of arguments");
+    }
+
     if (TYPE(argv[0]) == T_ARRAY) {
 	if (RARRAY(argv[0])->len != 2) {
@@ -620,4 +627,9 @@ rb_f_system(argc, argv)
     int state;
 
+    if (argc == 0) {
+	rb_last_status = INT2FIX(0);
+	rb_raise(rb_eArgError, "wrong # of arguments");
+    }
+
     if (TYPE(argv[0]) == T_ARRAY) {
 	if (RARRAY(argv[0])->len != 2) {
@@ -668,5 +680,5 @@ rb_f_system(argc, argv)
     rb_notimplement();
 #else
-   volatile VALUE prog = 0;
+    volatile VALUE prog = 0;
     int pid;
     int i;