81 lines
2.3 KiB
Diff
81 lines
2.3 KiB
Diff
|
From fc5f3468dcbee38eb202cfd552a5b8dbff990c7b Mon Sep 17 00:00:00 2001
|
||
|
|
From: Tony Cook <tony@develop-help.com>
|
||
|
|
Date: Tue, 12 May 2020 10:59:08 +1000
|
||
|
|
Subject: [PATCH 2/2] IO::Handle: clear the error on both input and output
|
||
|
|
streams
|
||
|
|
MIME-Version: 1.0
|
||
|
|
Content-Type: text/plain; charset=UTF-8
|
||
|
|
Content-Transfer-Encoding: 8bit
|
||
|
|
|
||
|
|
Similarly to GH #6799 clearerr() only cleared the error status
|
||
|
|
of the input stream, so clear both.
|
||
|
|
|
||
|
|
Signed-off-by: Petr Písař <ppisar@redhat.com>
|
||
|
|
---
|
||
|
|
dist/IO/IO.xs | 14 +++++++++++---
|
||
|
|
dist/IO/t/io_xs.t | 8 +++++---
|
||
|
|
2 files changed, 16 insertions(+), 6 deletions(-)
|
||
|
|
|
||
|
|
diff --git a/dist/IO/IO.xs b/dist/IO/IO.xs
|
||
|
|
index 99d523d2c1..9158106416 100644
|
||
|
|
--- a/dist/IO/IO.xs
|
||
|
|
+++ b/dist/IO/IO.xs
|
||
|
|
@@ -410,13 +410,21 @@ ferror(handle)
|
||
|
|
|
||
|
|
int
|
||
|
|
clearerr(handle)
|
||
|
|
- InputStream handle
|
||
|
|
+ SV * handle
|
||
|
|
+ PREINIT:
|
||
|
|
+ IO *io = sv_2io(handle);
|
||
|
|
+ InputStream in = IoIFP(io);
|
||
|
|
+ OutputStream out = IoOFP(io);
|
||
|
|
CODE:
|
||
|
|
if (handle) {
|
||
|
|
#ifdef PerlIO
|
||
|
|
- PerlIO_clearerr(handle);
|
||
|
|
+ PerlIO_clearerr(in);
|
||
|
|
+ if (in != out)
|
||
|
|
+ PerlIO_clearerr(out);
|
||
|
|
#else
|
||
|
|
- clearerr(handle);
|
||
|
|
+ clearerr(in);
|
||
|
|
+ if (in != out)
|
||
|
|
+ clearerr(out);
|
||
|
|
#endif
|
||
|
|
RETVAL = 0;
|
||
|
|
}
|
||
|
|
diff --git a/dist/IO/t/io_xs.t b/dist/IO/t/io_xs.t
|
||
|
|
index f890e92558..a8833b0651 100644
|
||
|
|
--- a/dist/IO/t/io_xs.t
|
||
|
|
+++ b/dist/IO/t/io_xs.t
|
||
|
|
@@ -11,7 +11,7 @@ BEGIN {
|
||
|
|
}
|
||
|
|
}
|
||
|
|
|
||
|
|
-use Test::More tests => 7;
|
||
|
|
+use Test::More tests => 8;
|
||
|
|
use IO::File;
|
||
|
|
use IO::Seekable;
|
||
|
|
|
||
|
|
@@ -58,12 +58,14 @@ SKIP: {
|
||
|
|
# This isn't really a Linux/BSD specific test, but /dev/full is (I
|
||
|
|
# hope) reasonably well defined on these. Patches welcome if your platform
|
||
|
|
# also supports it (or something like it)
|
||
|
|
- skip "no /dev/full or not a /dev/full platform", 2
|
||
|
|
+ skip "no /dev/full or not a /dev/full platform", 3
|
||
|
|
unless $^O =~ /^(linux|netbsd|freebsd)$/ && -c "/dev/full";
|
||
|
|
open my $fh, ">", "/dev/full"
|
||
|
|
- or skip "Could not open /dev/full: $!", 2;
|
||
|
|
+ or skip "Could not open /dev/full: $!", 3;
|
||
|
|
$fh->print("a" x 1024);
|
||
|
|
ok(!$fh->flush, "should fail to flush");
|
||
|
|
ok($fh->error, "stream should be in error");
|
||
|
|
+ $fh->clearerr;
|
||
|
|
+ ok(!$fh->error, "check clearerr removed the error");
|
||
|
|
close $fh; # silently ignore the error
|
||
|
|
}
|
||
|
|
--
|
||
|
|
2.25.4
|
||
|
|
|