| commit | b763133190b8b6c39837b34b6ef1e2b7d357e2b9 | |
| author | Dan Bolser <dan.bolser@gmail.com> | |
| Wed, 18 Aug 2010 14:19:11 +0000 (18 22:19 +0800) | ||
| committer | Chris Fields <cjfields@bioperl.org> | |
| Wed, 18 Aug 2010 16:53:37 +0000 (19 00:53 +0800) | ||
| tree | 435b612918eac18d65658d00b9a1e73082330acc | treesnapshot (tar.gz zip) |
| parent | d5baa73f92e40b223b054e4fdc32c522f8b70cb0 | commitdiff |
The use of $stop here is inconsistent, using $end.
After reading this I accidentally created code like this:
my ($start, $end, $strand) =
Bio::Range->union( @ranges );
my $newrange =
Bio::Range->new( -name => $chr,
-start => $start,
-stop => $stop,
);
Clearly wrong, but easy to do. I've changed $stop to $end in the documentation for consistency with the $end range parameter.
The error I was seeing was going un-caught, but I haven't fixed that issue with this commit:
Use of uninitialized value in sort at /homes/dbolser/perl5/lib/perl5/Bio/RangeI.pm line 411.
After reading this I accidentally created code like this:
my ($start, $end, $strand) =
Bio::Range->union( @ranges );
my $newrange =
Bio::Range->new( -name => $chr,
-start => $start,
-stop => $stop,
);
Clearly wrong, but easy to do. I've changed $stop to $end in the documentation for consistency with the $end range parameter.
The error I was seeing was going un-caught, but I haven't fixed that issue with this commit:
Use of uninitialized value in sort at /homes/dbolser/perl5/lib/perl5/Bio/RangeI.pm line 411.
| Bio/RangeI.pm | diffblobblamehistory |