From 90991d9889a7b95d85c13bdea3eb246d27e466ac Mon Sep 17 00:00:00 2001
From: matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Thu, 7 Feb 2002 04:18:08 +0000
Subject: [PATCH] * io.c (io_write): flag when buffered write is done.

* io.c (fptr_finalize): do not raise error on EBADF if write
  buffer is empty.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@2052 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 ChangeLog |  7 +++++++
 io.c      | 28 ++++++++++++++++++++++++----
 rubyio.h  |  1 +
 3 files changed, 32 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 7d6887004a..07b3257add 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Thu Feb  7 00:18:43 2002  Yukihiro Matsumoto  <matz@ruby-lang.org>
+
+	* io.c (io_write): flag when buffered write is done.
+
+	* io.c (fptr_finalize): do not raise error on EBADF if write
+	  buffer is empty.
+
 Wed Feb  6 17:18:54 2002  Nobuyoshi Nakada  <nobu.nakada@nifty.ne.jp>
 
 	* configure.in: keep old config.h unless changed.
diff --git a/io.c b/io.c
index 7ebc3b454b..9f1550812a 100644
--- a/io.c
+++ b/io.c
@@ -268,6 +268,10 @@ io_write(io, str)
 #endif
     if (fptr->mode & FMODE_SYNC) {
 	io_fflush(f, fptr->path);
+	fptr->mode &= ~FMODE_WBUF;
+    }
+    else {
+	fptr->mode |= FMODE_WBUF;
     }
 
     return INT2FIX(n);
@@ -300,6 +304,7 @@ rb_io_flush(io)
     f = GetWriteFile(fptr);
     
     io_fflush(f, fptr->path);
+    fptr->mode &= ~FMODE_WBUF;
 
     return io;
 }
@@ -451,6 +456,7 @@ rb_io_fsync(io)
     f = GetWriteFile(fptr);
     
     io_fflush(f, fptr->path);
+    fptr->mode &= ~FMODE_WBUF;
     if (fsync(fileno(f)) < 0)
 	rb_sys_fail(fptr->path);
     return INT2FIX(0);
@@ -1103,13 +1109,18 @@ fptr_finalize(fptr, fin)
     if (fptr->f2) {
 	f2 = fileno(fptr->f2);
 	n2 = fclose(fptr->f2);
+	fptr->f2 = 0;
 	if (n2 < 0) e = errno;
     }
-    if (fptr->f && fptr->f != fptr->f2) {
+    if (fptr->f) {
 	f1 = fileno(fptr->f);
 	n1 = fclose(fptr->f);
-	if (n1 < 0 && (e = errno) == EBADF && f1 == f2)
-	    n1 = 0;
+	fptr->f = 0;
+	if (n1 < 0 && errno == EBADF) {
+	    if (f1 == f2 || !(fptr->mode & FMODE_WBUF)) {
+		n1 = 0;
+	    }
+	}
     }
     if (!fin && (n1 < 0 || n2 < 0)) {
 	if (n1 == 0) errno = e;
@@ -1128,7 +1139,6 @@ rb_io_fptr_cleanup(fptr, fin)
     else {
 	fptr_finalize(fptr, fin);
     }
-    fptr->f = fptr->f2 = 0;
 
     if (fptr->path) {
 	free(fptr->path);
@@ -1259,6 +1269,9 @@ rb_io_syswrite(io, str)
     rb_io_check_writable(fptr);
     f = GetWriteFile(fptr);
 
+    if (fptr->mode & FMODE_WBUF) {
+	rb_warn("syswrite for buffered IO");
+    }
     if (!rb_thread_fd_writable(fileno(f))) {
         rb_io_check_closed(fptr);
     }
@@ -1975,6 +1988,7 @@ io_reopen(io, nfile)
     else if (orig->mode & FMODE_WRITABLE) {
 	io_fflush(orig->f, orig->path);
     }
+    orig->mode &= ~FMODE_WBUF;
     rb_thread_fd_close(fileno(fptr->f));
 
     /* copy OpenFile structure */
@@ -2104,6 +2118,7 @@ rb_io_clone(io)
     else if (orig->mode & FMODE_WRITABLE) {
 	io_fflush(orig->f, orig->path);
     }
+    orig->mode &= ~FMODE_WBUF;
 
     /* copy OpenFile structure */
     fptr->mode = orig->mode;
@@ -2228,8 +2243,13 @@ rb_io_putc(io, ch)
 
     if (fputc(c, f) == EOF)
 	rb_sys_fail(fptr->path);
+    fptr->mode |= FMODE_WBUF;
     if (fptr->mode & FMODE_SYNC) {
 	io_fflush(f, fptr->path);
+	fptr->mode &= ~FMODE_WBUF;
+    }
+    else {
+	fptr->mode |= FMODE_WBUF;
     }
 
     return ch;
diff --git a/rubyio.h b/rubyio.h
index fb012045f1..214128449f 100644
--- a/rubyio.h
+++ b/rubyio.h
@@ -31,6 +31,7 @@ typedef struct OpenFile {
 #define FMODE_READWRITE 3
 #define FMODE_BINMODE   4
 #define FMODE_SYNC      8
+#define FMODE_WBUF     16
 
 #define GetOpenFile(obj,fp) rb_io_check_closed((fp) = RFILE(rb_io_taint_check(obj))->fptr)