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

Report information
The Basics
Id:
20650
Status:
resolved
Priority:
Low/Low

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

BugTracker
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>
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 runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

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