Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug

2013-01-18 Thread Ricardo Signes
* Marc Green pongu...@gmail.com [2012-02-19T16:19:08]
 When given the following input, Pod::Simple does not warn that the [text]
 portion of the 2nd and 3rd =item is invalid.
 
 =over
 =item a definition
 some text
 =item *
 a bullet
 =item 1
 a number
 =back
 
 [perlpodspec cited]
 
 I am not sure why it doesn't warn in this situation, but does warn for
 other =item type mismatches.

This seemed to go nowhere, but was brought up to help get Pod::Checker using
Pod::Simple.

I hope we can agree that if the test results bit gets sorted out, we're in
favor of this warning..?

-- 
rjbs


signature.asc
Description: Digital signature


Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug

2013-01-18 Thread David E. Wheeler
On Jan 18, 2013, at 7:31 AM, Ricardo Signes perl@rjbs.manxome.org wrote:

 I hope we can agree that if the test results bit gets sorted out, we're in
 favor of this warning..?
 
 I have submitted https://github.com/theory/pod-simple/pull/44

+1. Will merge it if there are no objections.

 I also had the idea, mentioned in the commits, that we could consider 
 replacing
 the XML output for testing with JSON output.  This would make it easier to 
 test
 subsections and identify structural differences, since we have a JSON parser 
 in
 core but not an XML parser.

Good idea, though it will require that older versions of Perl install a JSON 
parser just to run the tests…

David



Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug

2013-01-18 Thread Ricardo Signes
* David E. Wheeler da...@justatheory.com [2013-01-18T12:33:52]
 Good idea, though it will require that older versions of Perl install a JSON
 parser just to run the tests…

I believe it could be bundled, and would be worth the hassle.  Maybe. :-)

-- 
rjbs


signature.asc
Description: Digital signature


Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug

2013-01-18 Thread Karl Williamson

On 01/18/2013 10:33 AM, David E. Wheeler wrote:

On Jan 18, 2013, at 7:31 AM, Ricardo Signes perl@rjbs.manxome.org wrote:


I hope we can agree that if the test results bit gets sorted out, we're in
favor of this warning..?


I have submitted https://github.com/theory/pod-simple/pull/44


+1. Will merge it if there are no objections.


+1



Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug

2013-01-18 Thread David E. Wheeler
On Jan 18, 2013, at 10:09 AM, Ricardo Signes perl@rjbs.manxome.org wrote:

 I believe it could be bundled, and would be worth the hassle.  Maybe. :-)

Well, if it's a hassle someone else wants, great! :-)

David