Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the PPI CPAN distribution.

Report information
The Basics
Id: 30863
Status: resolved
Priority: 0/
Queue: PPI

People
Owner: Nobody in particular
Requestors: sjq-perl [...] jadevine.org.uk
Cc:
AdminCc:

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



Subject: Documentation errors
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1195560683-3929-1"
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 184
Download (untitled) / with headers
text/plain 184b
I spotted a few errors in the docs which make some of the examples a little misleading. Attached is a patch against version 1.202_01 which fixes the mistakes. Cheers, Stephen Quinney
Subject: PPI.patch
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="----------=_1195560679-3929-0"
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 0
Content-Type: text/x-patch; name="PPI.patch"
Content-Disposition: inline; filename="PPI.patch"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: ascii
Content-Length: 2184
Download PPI.patch
text/x-diff 2.1k
diff -ru PPI-1.202_01.orig/lib/PPI/Element.pm PPI-1.202_01/lib/PPI/Element.pm --- PPI-1.202_01.orig/lib/PPI/Element.pm 2007-11-20 03:56:26.000000000 +0000 +++ PPI-1.202_01/lib/PPI/Element.pm 2007-11-20 12:01:52.000000000 +0000 @@ -150,7 +150,7 @@ =head2 statement For a C<PPI::Element> that is contained (at some depth) within a -L<PPI::Statment>, the C<statement> method will return the first parent +L<PPI::Statement>, the C<statement> method will return the first parent Statement object lexically 'above' the Element. Returns a L<PPI::Statement> object, which may be the same Element if the diff -ru PPI-1.202_01.orig/lib/PPI.pm PPI-1.202_01/lib/PPI.pm --- PPI-1.202_01.orig/lib/PPI.pm 2007-11-20 03:56:26.000000000 +0000 +++ PPI-1.202_01/lib/PPI.pm 2007-11-20 11:57:55.000000000 +0000 @@ -536,7 +536,7 @@ PPI::Token::Comment '#!/usr/bin/perl\n' PPI::Token::Whitespace '\n' PPI::Statement::Expression - PPI::Token::Bareword 'print' + PPI::Token::Word 'print' PPI::Structure::List ( ... ) PPI::Token::Whitespace ' ' PPI::Statement::Expression @@ -546,7 +546,7 @@ PPI::Token::Whitespace '\n' PPI::Token::Whitespace '\n' PPI::Statement::Expression - PPI::Token::Bareword 'exit' + PPI::Token::Word 'exit' PPI::Structure::List ( ... ) PPI::Token::Structure ';' PPI::Token::Whitespace '\n' @@ -563,13 +563,13 @@ PPI::Document PPI::Token::Comment '#!/usr/bin/perl\n' PPI::Statement::Expression - PPI::Token::Bareword 'print' + PPI::Token::Word 'print' PPI::Structure::List ( ... ) PPI::Statement::Expression PPI::Token::Quote::Double '"Hello World!"' PPI::Token::Structure ';' PPI::Statement::Expression - PPI::Token::Bareword 'exit' + PPI::Token::Word 'exit' PPI::Structure::List ( ... ) PPI::Token::Structure ';'
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-31149-1415796657-1979.30863-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 74
The full version of the fix for this is now released to CPAN as PPI 1.218.


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

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.