Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Pod-Simple CPAN distribution.

Report information
The Basics
Id: 93792
Status: resolved
Priority: 0/
Queue: Pod-Simple
People
Owner: Nobody in particular
Requestors: nigelm [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 3.28
Fixed in: 3.29

History Show all quoted text
  Wed Mar 12 10:30:25 2014 nigelm [...] cpan.org - Ticket created
Subject: Pod::Simple::Text sets Text::Wrap overflow policy wrong
In Pod/Simple/Text.pm the overflow policy for Text::Wrap is set using $Text::Wrap::wrap = 'overflow'; However the variable used for this purpose within Text::Wrap is $huge, so that line ought to be $Text::Wrap::huge = 'overflow'; This is the appropriate setting for Text::Wrap versions 2012.0818 and 2013.0523 - I guess it may have change in the dim and distant past, so maybe having both set might be a possibility... If you have stupidly long words within the Pod, the result of this bug is for a rather unhelpful error message to be emitted from Text::Wrap ('This shouldn't happen').
  Wed Mar 12 14:00:15 2014 dwheeler [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #93792] Pod::Simple::Text sets Text::Wrap overflow policy wrong
Date: Wed, 12 Mar 2014 10:59:56 -0700
To: bug-Pod-Simple [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
On Mar 12, 2014, at 7:30 AM, nigelm@cpan.org via RT <bug-Pod-Simple@rt.cpan.org> wrote: Show quoted text
> $Text::Wrap::wrap = 'overflow'; > > However the variable used for this purpose within Text::Wrap is $huge, so that line ought to be > > $Text::Wrap::huge = 'overflow';
Hrm. Should it use both, then? D
  Wed Mar 12 14:00:16 2014 The RT System itself - Status changed from 'new' to 'open'
  Thu Mar 13 06:57:44 2014 nigelm [...] cpan.org - Correspondence added
From: nigelm [...] cpan.org
On Wed Mar 12 14:00:15 2014, DWHEELER wrote: Show quoted text
> Hrm. Should it use both, then?
I've worked through the release sets of Text::Wrap and found that the variable to control how things behave when a word is too big to wrap has always been $Text::Wrap::huge since the time the feature was first implemented - see https://github.com/gitpan/Text-Tabs-Wrap/commit/6a0caf367fe5aad321959353a818c1b412b21221 The default behaviour (both before and after the implementation of the huge control variable) has changed a few times - flipping between dieing and passing it through. At present there it appears the default can be to die. Suggest the use of $Text::Wrap::wrap be changed to $Text::Wrap::huge since the former implementation will have never worked AFAIK.
  Thu Mar 13 13:16:39 2014 dwheeler [...] cpan.org - Correspondence added
Fixed in https://github.com/theory/pod-simple/commit/de57263b551c31d9193494bf519a2fa9b7c2257e. Thanks!
  Thu Mar 13 13:16:40 2014 dwheeler [...] cpan.org - Status changed from 'open' to 'patched'
  Wed Jan 14 13:44:54 2015 dwheeler [...] cpan.org - Status changed from 'patched' to 'resolved'
  Wed Jan 14 13:44:55 2015 dwheeler [...] cpan.org - Fixed in 3.29 added