Skip Menu |

This queue is for tickets about the Text-Levenshtein-Damerau-XS CPAN distribution.

Report information
The Basics
Id: 85523
Status: resolved
Priority: 0/
Queue: Text-Levenshtein-Damerau-XS
People
Owner: UGEXE [...] cpan.org
Requestors: m-uchino [...] yetipapa.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 2.8
Fixed in: 2.9

History Show all quoted text
  Tue May 21 16:04:55 2013 m-uchino [...] yetipapa.com - Ticket created
Subject: xs_edistance() 3rd argument
Hello Thank you for your good module. I have a question. I thought that following all return 1. perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print xs_edistance("xxx","xxxx",1),"\n";' -1 perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print xs_edistance("xxx","xxxx",2),"\n";' -1 perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print xs_edistance("xxx","xxxx",3),"\n";' 1 score is 1, and argument(limit) is more, then result is 1. Did I get a wrong anything? (Sorry my english is wrong)
  Wed May 22 17:53:03 2013 UGEXE [...] cpan.org - Correspondence added
Thanks for bringing this to my attention. I have released v2.9, which will return the correct results (I have added your examples to t/xs_edistance.t). On Tue May 21 16:04:55 2013, m-uchino@yetipapa.com wrote: Show quoted text
> Hello > Thank you for your good module. > > I have a question. I thought that following all return 1. > > perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print > xs_edistance("xxx","xxxx",1),"\n";' > -1 > perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print > xs_edistance("xxx","xxxx",2),"\n";' > -1 > perl -e'use Text::Levenshtein::Damerau::XS qw/xs_edistance/;print > xs_edistance("xxx","xxxx",3),"\n";' > 1 > > score is 1, and argument(limit) is more, then result is 1. > Did I get a wrong anything? > > (Sorry my english is wrong)
  Wed May 22 17:53:03 2013 The RT System itself - Status changed from 'new' to 'open'
  Wed May 22 17:53:03 2013 UGEXE [...] cpan.org - Status changed from 'open' to 'patched'
  Wed May 22 17:53:03 2013 UGEXE [...] cpan.org - Taken
  Wed May 22 17:53:03 2013 UGEXE [...] cpan.org - Fixed in 2.9 added
  Wed May 22 18:25:27 2013 UGEXE [...] cpan.org - Status changed from 'patched' to 'resolved'