[ruby-dev:24967] Re: sync and stdio buffering

まつもと ゆきひろです

In message "Re: [ruby-dev:24966] Re: sync and stdio buffering"
    on Sat, 27 Nov 2004 04:24:50 +0900, Tanaka Akira <[e-mail]> writes:

|sync なときには、rb_io_fwrite で fflush + write(2) で出力するというも
|のです。write(2) で直接出力すれば、データは stdio のバッファを経由しな
|いので、fflush で出力しようとして蒸発するのを避けられます。

十分に理解していないような気もするのですが、以下のようなコー
ドを想定しておられますか？

--- io.c	27 Nov 2004 02:18:12 -0000	1.331
+++ io.c	27 Nov 2004 03:12:40 -0000
@@ -400,10 +400,15 @@ rb_io_wait_writable(f)
 long
-rb_io_fwrite(ptr, len, f)
+io_fwrite(ptr, len, fptr)
     const char *ptr;
     long len;
-    FILE *f;
+    OpenFile *fptr;
 {
     long n, r;
+    FILE *f = GetWriteFile(fptr);
 
     if ((n = len) <= 0) return n;
+    if (fptr->mode & FMODE_SYNC) {
+	io_fflush(f, fptr);
+	return write(fileno(f), ptr, len);
+    }
 #if defined __human68k__
@@ -426,2 +431,3 @@ rb_io_fwrite(ptr, len, f)
 	    if (rb_io_wait_writable(fileno(f))) {
+		rb_io_check_closed(fptr);
 		clearerr(f);
@@ -436,2 +442,16 @@ rb_io_fwrite(ptr, len, f)
 
+long
+rb_io_fwrite(ptr, len, f)
+    const char *ptr;
+    long len;
+    FILE *f;
+{
+    OpenFile of;
+
+    of.f = f;
+    of.f2 = NULL;
+    of.mode = FMODE_WRITABLE;
+    return io_fwrite(ptr, len, &of);
+}
+
 /*
@@ -475,12 +495,8 @@ io_write(io, str)
     rb_io_check_writable(fptr);
-    f = GetWriteFile(fptr);
 
     rb_str_locktmp(str);
-    n = rb_io_fwrite(RSTRING(str)->ptr, RSTRING(str)->len, f);
+    n = io_fwrite(RSTRING(str)->ptr, RSTRING(str)->len, fptr);
     rb_str_unlocktmp(str);
     if (n == -1L) rb_sys_fail(fptr->path);
-    if (fptr->mode & FMODE_SYNC) {
-	io_fflush(f, fptr);
-    }
-    else {
+    if (!(fptr->mode & FMODE_SYNC)) {
 	fptr->mode |= FMODE_WBUF;