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>
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.