| Next | Program Repair Shop | 16 | 
     297  print STDERR "jsjv: find is generating the list of files\n"
     298      if ($verbose1);
This pattern appears several times throughout the program
We should consider using a subroutine to see if it makes the code easier to read
Something like this:
        verbose "jsjv: find is generating the list of files\n";
Yes, I'd say so
Here's verbose():
        sub verbose {
          return unless $opt_v;
          print STDERR @_;
        }
| Next | ![]()  | 
    Copyright © 2006 M. J. Dominus |