Re: Pod::Simple can treat binary as pod due to liberal/inconsistent regexp patterns

On 01/13/2015 10:15 PM, David E. Wheeler wrote:
> On Jan 12, 2015, at 11:42 AM, David E. Wheeler <[email protected]> wrote:
>
>> Honest, since the current regex matches stuff that is not in fact Pod, I think it is reasonable to tighten up the regex to
>>
>>     /\A=([a-zA-Z]+[0=9]*)\b/
>
> That one, it turns out, was no less liberal than the previous regex. I added a test matching the pattern Randy identified, and it failed with this regex, too. So I instead copied the regex from later in the file, which *is* sufficiently more strict, and brings them into line, to boot. The change is here:
>
>    https://github.com/theory/pod-simple/commit/31942ec
>
> Look good? If so, I will update perlpodspec to match it and send it off to p5p.
>
> Best,
>
> David
>

Nobody has explained to me why we accept uppercase when all existing pod 
commands (I believe) are entirely lowercase.  And shouldn't the digits 
only be in the final position?  I don't know the answers, but am just 
pointing out potential problems.