Skip Menu |

This queue is for tickets about the FileHandle-Unget CPAN distribution.

Report information
The Basics
Id: 48528
Status: resolved
Priority: 0/
Queue: FileHandle-Unget

People
Owner: Nobody in particular
Requestors: whn [...] lopi.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



CC: "Bryan H. Apple" <Bryan.Apple [...] recall.com>
Subject: Bug in FileHandle::Ungetc Ver. 0.16.22 on line 394
Date: Fri, 7 Aug 2009 15:42:52 -0400
To: bug-FileHandle-Unget [...] rt.cpan.org
From: Bill Nugent <whn [...] lopi.com>
Howdy, Wonderful module but unfortunately I found a bug while reading the output from the program pdftotext. I have a pdf file than has as the last page an image so the program is printing out two formfeeds ('0x0c') as the last two characters. FileHandle::Unget gets confused. Included is a short perl program and a text file that tickles this bug. One way to fix this bug is to change line 392 from: else to: elif (defined $template) which eliminates the attempt to append the $templine which has an undef value because the file has reached the end. I've included a patch. Thank you, Bill

Message body is not shown because sender requested not to inline it.

Message body is not shown because sender requested not to inline it.

Message body is not shown because sender requested not to inline it.

Thanks for the patch. Note that you have a bug in it--$template is not defined. :) But the idea of the patch is right. I'll roll it out with versions > 0.1621
Subject: Re: [rt.cpan.org #48528] Bug in FileHandle::Ungetc Ver. 0.16.22 on line 394
Date: Sun, 9 Aug 2009 19:22:00 -0400
To: bug-FileHandle-Unget [...] rt.cpan.org
From: Bill Nugent <whn [...] lopi.com>
Ha ha - joke's on me. My measly excuse: I'd been working with templates for several hours...sigh. :-) Thank you again for a great module - you can see what was on my mind! Bill On Sunday 09 August 2009 at 17:02, David Coppit via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=48528 > > > Thanks for the patch. Note that you have a bug in it--$template is > not defined. :) > > But the idea of the patch is right. I'll roll it out with versions > > 0.1621