summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog15
-rw-r--r--io.c15
2 files changed, 18 insertions, 12 deletions
diff --git a/ChangeLog b/ChangeLog
index aec10fe4d8..8549261a24 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Tue Jan 6 21:51:37 2004 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * io.c (fptr_finalize): should save errno just after failure.
+ [ruby-dev:22492]
+
Tue Jan 6 20:51:10 2004 NAKAMURA, Hiroshi <nakahiro@sarion.co.jp>
* lib/logger.rb(Logger#msg2str): no special treatment for the object
@@ -75,7 +80,7 @@ Tue Jan 6 06:37:53 2004 Dave Thomas <dave@pragprog.com>
Tue Jan 6 00:04:40 2004 Dave Thomas <dave@pragprog.com>
- * lib/rdoc/parsers/parse_rb.rb (RDoc::RubyParser::parse_method_or_yield_parameters):
+ * lib/rdoc/parsers/parse_rb.rb (RDoc::RubyParser::parse_method_or_yield_parameters):
fix parsing if there are braces in a method parameter list
Tue Jan 6 01:01:04 2004 NAKAMURA Usaku <usa@ruby-lang.org>
@@ -96,7 +101,7 @@ Mon Jan 5 18:58:47 2004 Yukihiro Matsumoto <matz@ruby-lang.org>
* pack.c (pack_unpack): unpack requires big endian offet (OFF16B
and OFF32B). The patch is from Minero Aoki in [ruby-dev:22489]
- * pack.c (OFF16B): add big-endian offset again.
+ * pack.c (OFF16B): add big-endian offset again.
Mon Jan 5 03:00:53 2004 Minero Aoki <aamine@loveruby.net>
@@ -118,7 +123,7 @@ Sat Jan 3 01:18:08 2004 Yukihiro Matsumoto <matz@ruby-lang.org>
[ruby-dev:22476]
Fri Jan 2 14:54:11 2004 Dave Thomas <dave@pragprog.com>
-
+
* bin/ri: Add new --classes option, and arrange for
help messages to be paged too.
@@ -126,7 +131,7 @@ Fri Jan 2 14:54:11 2004 Dave Thomas <dave@pragprog.com>
* process.c: (MG) Added Process documentation
- * lib/rdoc/ri/ri_formatter.rb (RI::AttributeFormatter::wrap):
+ * lib/rdoc/ri/ri_formatter.rb (RI::AttributeFormatter::wrap):
Fix problem with labels not displaying in RI labeled
lists using BS and ANSI modes.
@@ -266,7 +271,7 @@ Sun Dec 28 08:56:51 2003 Dave Thomas <dave@pragprog.com>
Sun Dec 28 03:50:05 2003 Dave Thomas <dave@pragprog.com>
- * lib/rdoc/parsers/parse_c.rb (RDoc::C_Parser::find_override_comment):
+ * lib/rdoc/parsers/parse_c.rb (RDoc::C_Parser::find_override_comment):
Escape method names used in regexp
Sun Dec 28 01:46:02 2003 Dave Thomas <dave@wireless_3.local.thomases.com>
diff --git a/io.c b/io.c
index fba74b8336..f49fce3685 100644
--- a/io.c
+++ b/io.c
@@ -1727,12 +1727,12 @@ fptr_finalize(fptr, noraise)
OpenFile *fptr;
int noraise;
{
- int n1 = 0, n2 = 0, e = 0, f1, f2 = -1;
+ int n1 = 0, n2 = 0, f1, f2 = -1;
if (fptr->f2) {
f2 = fileno(fptr->f2);
- while ((n2 = fclose(fptr->f2)) < 0) {
- e = errno;
+ while (n2 = 0, fclose(fptr->f2) < 0) {
+ n2 = errno;
if (!rb_io_wait_writable(f2)) {
break;
}
@@ -1742,18 +1742,19 @@ fptr_finalize(fptr, noraise)
}
if (fptr->f) {
f1 = fileno(fptr->f);
- while ((n1 = fclose(fptr->f)) < 0) {
+ while (n1 = 0, fclose(fptr->f) < 0) {
+ n1 = errno;
if (f2 != -1 || !(fptr->mode & FMODE_WBUF)) break;
if (!rb_io_wait_writable(f1)) break;
if (!fptr->f) break;
}
fptr->f = 0;
- if (n1 < 0 && errno == EBADF && f1 == f2) {
+ if (n1 == EBADF && f1 == f2) {
n1 = 0;
}
}
- if (!noraise && (n1 < 0 || n2 < 0)) {
- if (n1 == 0) errno = e;
+ if (!noraise && (n1 || n2)) {
+ errno = (n1 ? n1 : n2);
rb_sys_fail(fptr->path);
}
}