Skip Menu |

This queue is for tickets about the IO-Pager CPAN distribution.

Report information
The Basics
Id: 82351
Status: resolved
Priority: 0/
Queue: IO-Pager

Owner: Nobody in particular
Requestors: dwheeler [...]

Bug Information
Severity: Critical
Broken in: 0.30
Fixed in: 0.31

Download (untitled) / with headers
text/plain 579b
The switch to IO::Handle in v0.30 is great, except that the handle returned does not have a file associated with it! This patch fixes it: --- a/lib/IO/Pager/ +++ b/lib/IO/Pager/ @@ -16,7 +16,7 @@ sub new(;$) { # [FH], procedural return $_[1] if defined($_[2]) && $_[2] eq 'procedural'; #...fall back to IO::Handle for transparent OO programming - eval "require IO::Handle" && return new IO::Handle; + eval 'require IO::Handle' && return IO::Handle->new_from_fd(fileno $_[1], 'w'); } $!=$@, return 0 if $@ =~ 'pipe';
Subject: Re: [ #82351]
Date: Mon, 31 Dec 2012 15:36:57 -0500
To: bug-IO-Pager [...]
From: Jerrad Pierce <belg4mit [...]>
Download (untitled) / with headers
text/plain 146b
Thanks, I'd actually played around with that, but did not use it for some reason that I have since forgotten. Perhaps it failed one of the tests?
Subject: Re: [ #82351]
Date: Mon, 31 Dec 2012 13:00:46 -0800
To: bug-IO-Pager [...]
From: "David E. Wheeler" <dwheeler [...]>
Download (untitled) / with headers
text/plain 105b
All tests pass for me with that change. It could probably use a regression test before applying. David
Heya, any chance of a release soon with a fix for this? Thanks! David

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to