develooper Front page | perl.perl5.porters | Postings from March 2011

Re: [perl #85120] [PATCH] Fix podchecker warnings in Filter::Simple

Thread Next
From:
Michael Stevens
Date:
March 7, 2011 00:15
Subject:
Re: [perl #85120] [PATCH] Fix podchecker warnings in Filter::Simple
Message ID:
20110307081509.GA32335@eris.etla.org
On Sun, Mar 06, 2011 at 01:10:46PM -0800, Father Chrysostomos via RT wrote:
> On Mon Feb 28 13:44:45 2011, mstevens wrote:
> > ---
> >  dist/Filter-Simple/lib/Filter/Simple.pm |   15 +++++++--------
> >  1 files changed, 7 insertions(+), 8 deletions(-)
> > 
> > diff --git a/dist/Filter-Simple/lib/Filter/Simple.pm b/dist/Filter-
> > Simple/lib/Filter/Simple.pm
> > index 2edd798..6893391 100644
> > --- a/dist/Filter-Simple/lib/Filter/Simple.pm
> > +++ b/dist/Filter-Simple/lib/Filter/Simple.pm
> > @@ -234,7 +234,6 @@ __END__
> > 
> >  Filter::Simple - Simplified source filtering
> > 
> > -
> >  =head1 SYNOPSIS
> > 
> >   # in MyFilter.pm:
> > @@ -242,7 +241,7 @@ Filter::Simple - Simplified source filtering
> >       package MyFilter;
> > 
> >       use Filter::Simple;
> > -
> > +
> >       FILTER { ... };
> 
> Isn’t that indent intentional, to make both lines part of the same
> verbatim paragraph? Some Pod formatters put verbatim paragraphs inside
> boxes. Removing the indent would cause this example to be broken up.
> 
> But, then, the line after the package declaration has no space, so it
> looks as though it needs to be fixed one way or the other, but I’m not
> sure which.
 
I think you may be right. Since I sent the original patch I've realised
this seems to be a difference between 5.10 podchecker and bleadperl
podchecker.

Michael

Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About