Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 65569
Status: resolved
Priority: 0/
Queue: Perl-Critic
People
Owner: Nobody in particular
Requestors: rjray [...] blackperl.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.111
Fixed in: 1.114

History Show all quoted text
  Tue Feb 08 14:16:07 2011 rjray [...] blackperl.com - Ticket created
Subject: Documentation::RequirePodLinksIncludeText gives false-positive on nested POD formatting
I have noticed that the referenced policy consistently breaks when there are nested POD format codes within the L<>. For example: L<C<perldiag>|perldiag> The regular expression used to match the content of the L<> format does not take into account nested formatting. Randy -- """"""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""" Randy J. Ray Silicon Valley Scale Modelers: http://www.svsm.org rjray@blackperl.com randy.j.ray@gmail.com
  Tue Feb 08 18:49:50 2011 wyant [...] cpan.org - Correspondence added
On Tue Feb 08 14:16:07 2011, RJRAY wrote: Show quoted text
> I have noticed that the referenced policy consistently breaks when there > are nested POD format codes within the L<>. For example: > > L<C<perldiag>|perldiag> > > The regular expression used to match the content of the L<> format does > not take into account nested formatting. > > Randy
Thank you for finding this. Nested formatting was probably not considered because you never know what a POD formatter is going to do with something out of the ordinary. This may be interesting to fix without Perl 5.10.
  Tue Feb 08 18:49:50 2011 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 15 19:52:56 2011 wyant [...] cpan.org - Correspondence added
Patch committed as SVN revision 4042. It uses Pod::Parser to do the heavy lifting, which is what the policy should have done in the first place.
  Tue Feb 15 19:52:57 2011 wyant [...] cpan.org - Status changed from 'open' to 'patched'
  Fri Apr 01 17:17:40 2011 wyant [...] cpan.org - Status changed from 'patched' to 'resolved'
  Fri Apr 01 17:17:40 2011 wyant [...] cpan.org - Fixed in 1.114 added