Next | Program Repair Shop | 20 |
verbose "jsjvv: md5sum found " . scalar (@{$file_name_by_md5sum{$md5sum}}) . " matches\n" if scalar (@{$file_name_by_md5sum{$md5sum}}) > 1;
Since we're here, lets deal with this small mess
What's really wrong here is the repetition of the complex expression
In fact, the corruption is more widespread than that:
644 foreach $md5sum (keys %file_name_by_md5sum) 645 { 646 push @md5_LoL , [ @{$file_name_by_md5sum{$md5sum}} ] 647 if ( scalar (@{$file_name_by_md5sum{$md5sum}}) > 1); 648 # do NOT push single file lists, there's nothing to match! 649 650 print STDERR "jsjvv: md5sum found " . 651 scalar (@{$file_name_by_md5sum{$md5sum}}) . " matches\n" 652 if ($verbose1) && ( scalar (@{$file_name_by_md5sum{$md5sum}}) > 1);
Of the visible characters in this code, 39% are in the repeated expression!
Next | ![]() |
Copyright © 2006 M. J. Dominus |