Skip Menu |

This queue is for tickets about the Text-Reform CPAN distribution.

Report information
The Basics
Id: 39958
Status: new
Priority: 0/
Queue: Text-Reform

People
Owner: Nobody in particular
Requestors: spamcollector_cpan [...] juerd.nl
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: v1.12.2
Fixed in: (no value)



Subject: Code refomatting with /x breaks existing code
The regexes in break_at got whitespace and /x; that does mean, however, that my interpolated $hyphen of " " (chr 32) now became meaningless. $hyphen needs a quotemeta; the documentation and the length($hyphen) suggest it's be used as a string, not a regex. These regexes were already readable. Adding /x, /m, or /s, for the sake of PBP compliancy, was (IMO) not a good idea. It can create new bugs, or like in this case expose an existing bug. -- Juerd