Skip Menu |
 

This queue is for tickets about the Algorithm-Diff CPAN distribution.

Report information
The Basics
Id: 20650
Status: resolved
Priority: 0/
Queue: Algorithm-Diff

People
Owner: Nobody in particular
Requestors: woodworthjames [...] yahoo.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: missing parameter to compare callback
Date: Mon, 24 Jul 2006 09:55:58 -0700 (PDT)
To: bug-Algorithm-Diff [...] rt.cpan.org
From: James Woodworth <woodworthjames [...] yahoo.com>
Download (untitled) / with headers
text/plain 552b
Hello, i found that at line 188 of Algorithm::DiffOld, the "@_" is missing; this is needed so that extra parameters into the "traverse_sequences" that are passed into _longestCommonSubsequence are subsequently passed into the callback in $compare. 188c188 next if ! &$compare( $a->[$i], $b->[$j] ); next if ! &$compare( $a->[$i], $b->[$j], @_ ); please correct me if i am wrong. thanks, jim w. --------------------------------- Talk is cheap. Use Yahoo! Messenger to make PC-to-Phone calls. Great rates starting at 1¢/min.
Fixed in 1.19_02, just uploaded. Thanks.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.