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

Owner: Nobody in particular
Requestors: woodworthjames [...]

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 [...]
From: James Woodworth <woodworthjames [...]>
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 infrastructure.

Please report any issues with to