Skip to content

Commit 74870bb

Browse files
committed
zap unused variables
1 parent e27d24a commit 74870bb

2 files changed

Lines changed: 16 additions & 23 deletions

File tree

lib/PDL/LinearAlgebra.pm

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1135,7 +1135,6 @@ sub PDL::mschur {
11351135
$mult //= 1;
11361136
$norm //= 1;
11371137
$jobv = $jobvl = $jobvr = 0 unless wantarray;
1138-
my $type = $m->type;
11391138
my $mm = $m->is_inplace ? $m->t : $m->t->copy;
11401139
my $v = $m->_similar_null;
11411140
my @w = map $m->_similar_null, $m->_is_complex ? 1 : 1..2;
@@ -1237,7 +1236,6 @@ sub PDL::mschurx {
12371236
$mult //= 1;
12381237
$norm //= 1;
12391238
$jobv = $jobvl = $jobvr = 0 unless wantarray;
1240-
my $type = $m->type;
12411239
my $select = long($select_func ? 1 : 0);
12421240
$sense = pdl(long,0) if !$select_func;
12431241
$_ = null for my ($info, $sdim, $rconde, $rcondv);
@@ -1393,7 +1391,6 @@ sub PDL::mgschur{
13931391
($jobvsl == 2 || $jobvsr == 2 || $jobvl == 2 || $jobvr == 2));
13941392
$mult //= 1;
13951393
$norm //= 1;
1396-
my $type = $m->type;
13971394
my $select = $select_func ? pdl(long,1) : pdl(long,0);
13981395
$_ = null for my ($info, $sdim);
13991396
my ($mm, $pp) = map $_->is_inplace ? $_->t : $_->t->copy, $m, $p;
@@ -1510,7 +1507,6 @@ sub PDL::mgschurx{
15101507
my (@mdims) = $m->dims;
15111508
$mult //= 1;
15121509
$norm //= 1;
1513-
my $type = $m->type;
15141510
my $select = $select_func ? 1 : 0;
15151511
$sense = 0 if !$select_func;
15161512
$_ = null for my ($info, $rconde, $rcondv, $sdim);
@@ -2284,7 +2280,6 @@ sub PDL::mlls {
22842280
my(@bdims) = $b->dims;
22852281
my $x;
22862282
$a = $a->copy;
2287-
my $type = $a->type;
22882283
if ( $adims[$di+1] < $adims[$di]) {
22892284
$x = $a->_similar($adims[$di], $bdims[$di]);
22902285
$x->slice("$slice_prefix:@{[$bdims[$di+1]-1]}, :@{[$bdims[$di]-1]}") .= $b->t;
@@ -2338,7 +2333,6 @@ sub PDL::mllsy {
23382333
my($a, $b) = @_;
23392334
my(@adims) = $a->dims;
23402335
my(@bdims) = $b->dims;
2341-
my $type = $a->type;
23422336
my $rcond = lamch(0);
23432337
$rcond = $rcond->sqrt - ($rcond->sqrt - $rcond) / 2;
23442338
$a = $a->t->copy;
@@ -3148,7 +3142,6 @@ sub PDL::mdsvd {
31483142
my $di = $_[0]->dims_internal;
31493143
my($m, $jobz) = @_;
31503144
my(@dims) = $m->dims;
3151-
my $type = $m->type;
31523145
$jobz = !wantarray ? 0 : $jobz // 1;
31533146
my $min = $dims[$di] > $dims[1+$di] ? $dims[1+$di]: $dims[$di];
31543147
$m = $m->copy;

t/common.pl

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -34,22 +34,22 @@
3434
runtest($a, 'mschurx', $schurx_soln, [0,2,2,undef,1,1]);
3535
runtest($a, 'mschurx', $schurx_soln, [0,2,2,sub {1},1,1]);
3636
runtest($a, 'mschurx', $schurx_soln, [0,2,2,sub {1},3,1]);
37-
my @mgschur_exp = (pdl([-0.35099581,-0.68880032],[0,0.81795847]),
38-
pdl([1.026674, -0.366662], [0, -0.279640]));
39-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2)]);
40-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2),1,1,1,1,sub {1}]);
41-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2),2,2,2,2,sub {1},0]);
42-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2),2,2,2,2,undef,0]);
43-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2),0,0,2,2,sub {1},1,0]);
44-
runtest($x, 'mgschur', \@mgschur_exp, [sequence(2,2),0,0,2,2,undef]);
45-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2)]);
46-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),1,1,1,1,sub {1}]);
47-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),2,2,2,2,sub {1},0,0,0]);
48-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),2,2,2,2,undef,0,0,0]);
49-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),1,1,1,1,sub {1},2]);
50-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),1,1,1,1,sub {1},3]);
51-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),0,0,1,1,sub {1},1,1,0]);
52-
runtest($x, 'mgschurx', \@mgschur_exp, [sequence(2,2),0,0,2,2,undef]);
37+
my $mgschur_exp = [pdl([-0.35099581,-0.68880032],[0,0.81795847]),
38+
pdl([1.026674, -0.366662], [0, -0.279640])];
39+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2)]);
40+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2),1,1,1,1,sub {1}]);
41+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2),2,2,2,2,sub {1},0]);
42+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2),2,2,2,2,undef,0]);
43+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2),0,0,2,2,sub {1},1,0]);
44+
runtest($x, 'mgschur', $mgschur_exp, [sequence(2,2),0,0,2,2,undef]);
45+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2)]);
46+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),1,1,1,1,sub {1}]);
47+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),2,2,2,2,sub {1},0,0,0]);
48+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),2,2,2,2,undef,0,0,0]);
49+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),1,1,1,1,sub {1},2]);
50+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),1,1,1,1,sub {1},3]);
51+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),0,0,1,1,sub {1},1,1,0]);
52+
runtest($x, 'mgschurx', $mgschur_exp, [sequence(2,2),0,0,2,2,undef]);
5353
runtest($x, 'mqr', pdl([-0.49738411,-0.86753043],[-0.86753043,0.49738411]));
5454
runtest($wide->t, 'mqr', pdl([-0.523069,-0.5023351],[-0.0364932,-0.793903],[-0.851508,0.34260173]));
5555
runtest($x, 'mrq', pdl([0.27614707,-0.3309725],[0,-1.0396634]));

0 commit comments

Comments
 (0)