Skip Menu |

This queue is for tickets about the Regexp-Common CPAN distribution.

Report information
The Basics
Id: 17974
Status: resolved
Priority: 0/
Queue: Regexp-Common
People
Owner: Nobody in particular
Requestors: husseinp [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 2.120
Fixed in: (no value)

History Show all quoted text
  Fri Mar 03 21:49:58 2006 Guest - Ticket created
Subject: Regexp::Common::number seems to have several mistakes
$RE{num}{decimal}->matches('asd.asd') returns true In fact so long as I have a period (or a valid radix) in the "number", it seems to return true.
  Sat Apr 16 10:25:30 2011 cpan [...] abigail.be - Correspondence added
On Fri Mar 03 21:49:58 2006, guest wrote: Show quoted text
> $RE{num}{decimal}->matches('asd.asd') returns true > > In fact so long as I have a period (or a valid radix) in the "number", > it seems to return true.
match() doesn't anchor the string. So, if there's a sub string that matches the pattern, the match succeeds. 'asd.asd' (used) to match because '.' matched. However, this is fixed in version 2011041601. But 'a1d.asd' will still match, because '1' matches.
  Sat Apr 16 10:25:31 2011 The RT System itself - Status changed from 'new' to 'open'
  Sat Apr 16 10:26:23 2011 cpan [...] abigail.be - Correspondence added
Partially solved in 2011011601. The other part of the report isn't a bug.
  Sat Apr 16 10:26:24 2011 cpan [...] abigail.be - Status changed from 'open' to 'resolved'