Mercurial > jwe > octave
annotate liboctave/numeric/CmplxSCHUR.cc @ 21136:7cac4e7458f2
maint: clean up code around calls to current_liboctave_error_handler.
Remove statements after call to handler that are no longer reachable.
Place input validation first and immediately call handler if necessary.
Change if/error_handler/else to if/error_handler and re-indent code.
* Array-util.cc, Array.cc, CColVector.cc, CDiagMatrix.cc, CMatrix.cc,
CNDArray.cc, CRowVector.cc, CSparse.cc, DiagArray2.cc, MArray.cc,
PermMatrix.cc, Sparse.cc, Sparse.h, chMatrix.cc, chNDArray.cc, dColVector.cc,
dDiagMatrix.cc, dMatrix.cc, dNDArray.cc, dRowVector.cc, dSparse.cc,
fCColVector.cc, fCDiagMatrix.cc, fCMatrix.cc, fCNDArray.cc, fCRowVector.cc,
fColVector.cc, fDiagMatrix.cc, fMatrix.cc, fNDArray.cc, fRowVector.cc,
idx-vector.cc, CmplxAEPBAL.cc, CmplxCHOL.cc, CmplxGEPBAL.cc, CmplxHESS.cc,
CmplxLU.cc, CmplxQR.cc, CmplxSCHUR.cc, CmplxSVD.cc, DASPK.cc, EIG.cc, LSODE.cc,
Quad.cc, SparseCmplxCHOL.cc, SparseCmplxLU.cc, SparseCmplxQR.cc, SparseQR.cc,
SparsedbleCHOL.cc, SparsedbleLU.cc, base-lu.cc, bsxfun-defs.cc, dbleAEPBAL.cc,
dbleCHOL.cc, dbleGEPBAL.cc, dbleHESS.cc, dbleLU.cc, dbleQR.cc, dbleSCHUR.cc,
dbleSVD.cc, eigs-base.cc, fCmplxAEPBAL.cc, fCmplxCHOL.cc, fCmplxLU.cc,
fCmplxQR.cc, fCmplxSCHUR.cc, fEIG.cc, floatAEPBAL.cc, floatCHOL.cc,
floatGEPBAL.cc, floatHESS.cc, floatLU.cc, floatQR.cc, floatSCHUR.cc,
floatSVD.cc, lo-specfun.cc, oct-fftw.cc, oct-rand.cc, oct-spparms.cc,
sparse-base-chol.cc, sparse-dmsolve.cc, file-ops.cc, lo-sysdep.cc,
mach-info.cc, oct-env.cc, oct-syscalls.cc, cmd-edit.cc, cmd-hist.cc,
data-conv.cc, lo-ieee.cc, lo-regexp.cc, oct-base64.cc, oct-shlib.cc,
pathsearch.cc, singleton-cleanup.cc, sparse-util.cc, unwind-prot.cc:
Remove statements after call to handler that are no longer reachable.
Place input validation first and immediately call handler if necessary.
Change if/error_handler/else to if/error_handler and re-indent code.
| author | Rik <rik@octave.org> |
|---|---|
| date | Sat, 23 Jan 2016 13:52:03 -0800 |
| parents | 4197fc428c7d |
| children | f7121e111991 |
| rev | line source |
|---|---|
| 457 | 1 /* |
| 2 | |
|
19697
4197fc428c7d
maint: Update copyright notices for 2015.
John W. Eaton <jwe@octave.org>
parents:
17744
diff
changeset
|
3 Copyright (C) 1994-2015 John W. Eaton |
| 457 | 4 |
| 5 This file is part of Octave. | |
| 6 | |
| 7 Octave is free software; you can redistribute it and/or modify it | |
| 8 under the terms of the GNU General Public License as published by the | |
| 7016 | 9 Free Software Foundation; either version 3 of the License, or (at your |
| 10 option) any later version. | |
| 457 | 11 |
| 12 Octave is distributed in the hope that it will be useful, but WITHOUT | |
| 13 ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 14 FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 15 for more details. | |
| 16 | |
| 17 You should have received a copy of the GNU General Public License | |
| 7016 | 18 along with Octave; see the file COPYING. If not, see |
| 19 <http://www.gnu.org/licenses/>. | |
| 457 | 20 |
| 21 */ | |
| 22 | |
| 23 #ifdef HAVE_CONFIG_H | |
| 1192 | 24 #include <config.h> |
| 457 | 25 #endif |
| 26 | |
| 27 #include "CmplxSCHUR.h" | |
| 1847 | 28 #include "f77-fcn.h" |
| 457 | 29 #include "lo-error.h" |
| 10822 | 30 #include "oct-locbuf.h" |
| 457 | 31 |
| 32 extern "C" | |
| 33 { | |
| 4552 | 34 F77_RET_T |
| 35 F77_FUNC (zgeesx, ZGEESX) (F77_CONST_CHAR_ARG_DECL, | |
|
10314
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
36 F77_CONST_CHAR_ARG_DECL, |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
37 ComplexSCHUR::select_function, |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
38 F77_CONST_CHAR_ARG_DECL, |
| 11495 | 39 const octave_idx_type&, Complex*, |
| 40 const octave_idx_type&, octave_idx_type&, | |
| 41 Complex*, Complex*, const octave_idx_type&, | |
| 42 double&, double&, Complex*, | |
| 43 const octave_idx_type&, double*, | |
| 44 octave_idx_type*, octave_idx_type& | |
|
10314
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
45 F77_CHAR_ARG_LEN_DECL |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
46 F77_CHAR_ARG_LEN_DECL |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
47 F77_CHAR_ARG_LEN_DECL); |
| 11495 | 48 |
| 10822 | 49 F77_RET_T |
| 11495 | 50 F77_FUNC (zrsf2csf, ZRSF2CSF) (const octave_idx_type&, Complex *, |
| 51 Complex *, double *, double *); | |
| 457 | 52 } |
| 53 | |
| 5275 | 54 static octave_idx_type |
| 1929 | 55 select_ana (const Complex& a) |
| 457 | 56 { |
| 1251 | 57 return a.real () < 0.0; |
| 457 | 58 } |
| 59 | |
| 5275 | 60 static octave_idx_type |
| 1929 | 61 select_dig (const Complex& a) |
| 457 | 62 { |
| 1251 | 63 return (abs (a) < 1.0); |
| 457 | 64 } |
| 65 | |
| 5275 | 66 octave_idx_type |
|
11586
12df7854fa7c
strip trailing whitespace from source files
John W. Eaton <jwe@octave.org>
parents:
11570
diff
changeset
|
67 ComplexSCHUR::init (const ComplexMatrix& a, const std::string& ord, |
|
10314
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
68 bool calc_unitary) |
| 457 | 69 { |
| 5275 | 70 octave_idx_type a_nr = a.rows (); |
| 71 octave_idx_type a_nc = a.cols (); | |
| 1929 | 72 |
| 457 | 73 if (a_nr != a_nc) |
|
21136
7cac4e7458f2
maint: clean up code around calls to current_liboctave_error_handler.
Rik <rik@octave.org>
parents:
19697
diff
changeset
|
74 (*current_liboctave_error_handler) ("ComplexSCHUR requires square matrix"); |
|
7cac4e7458f2
maint: clean up code around calls to current_liboctave_error_handler.
Rik <rik@octave.org>
parents:
19697
diff
changeset
|
75 |
|
7cac4e7458f2
maint: clean up code around calls to current_liboctave_error_handler.
Rik <rik@octave.org>
parents:
19697
diff
changeset
|
76 if (a_nr == 0) |
|
10607
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
77 { |
|
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
78 schur_mat.clear (); |
|
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
79 unitary_mat.clear (); |
|
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
80 return 0; |
|
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
81 } |
| 457 | 82 |
| 3334 | 83 // Workspace requirements may need to be fixed if any of the |
| 84 // following change. | |
| 85 | |
| 5008 | 86 char jobvs; |
| 1930 | 87 char sense = 'N'; |
| 88 char sort = 'N'; | |
| 1756 | 89 |
| 5008 | 90 if (calc_unitary) |
| 91 jobvs = 'V'; | |
| 92 else | |
| 93 jobvs = 'N'; | |
| 94 | |
| 1756 | 95 char ord_char = ord.empty () ? 'U' : ord[0]; |
| 96 | |
| 97 if (ord_char == 'A' || ord_char == 'D' || ord_char == 'a' || ord_char == 'd') | |
| 1930 | 98 sort = 'S'; |
| 457 | 99 |
| 1929 | 100 if (ord_char == 'A' || ord_char == 'a') |
| 101 selector = select_ana; | |
| 102 else if (ord_char == 'D' || ord_char == 'd') | |
| 103 selector = select_dig; | |
| 1930 | 104 else |
| 105 selector = 0; | |
| 457 | 106 |
| 5275 | 107 octave_idx_type n = a_nc; |
| 108 octave_idx_type lwork = 8 * n; | |
| 109 octave_idx_type info; | |
| 110 octave_idx_type sdim; | |
| 457 | 111 double rconde; |
| 112 double rcondv; | |
| 113 | |
| 1929 | 114 schur_mat = a; |
| 5008 | 115 if (calc_unitary) |
|
10607
f7501986e42d
make schur more Matlab compatible
Jaroslav Hajek <highegg@gmail.com>
parents:
10350
diff
changeset
|
116 unitary_mat.clear (n, n); |
| 1929 | 117 |
| 118 Complex *s = schur_mat.fortran_vec (); | |
| 119 Complex *q = unitary_mat.fortran_vec (); | |
| 120 | |
|
11570
57632dea2446
attempt better backward compatibility for Array constructors
John W. Eaton <jwe@octave.org>
parents:
11523
diff
changeset
|
121 Array<double> rwork (dim_vector (n, 1)); |
| 1929 | 122 double *prwork = rwork.fortran_vec (); |
| 123 | |
|
11570
57632dea2446
attempt better backward compatibility for Array constructors
John W. Eaton <jwe@octave.org>
parents:
11523
diff
changeset
|
124 Array<Complex> w (dim_vector (n, 1)); |
| 1929 | 125 Complex *pw = w.fortran_vec (); |
| 126 | |
|
11570
57632dea2446
attempt better backward compatibility for Array constructors
John W. Eaton <jwe@octave.org>
parents:
11523
diff
changeset
|
127 Array<Complex> work (dim_vector (lwork, 1)); |
| 1929 | 128 Complex *pwork = work.fortran_vec (); |
| 457 | 129 |
| 3334 | 130 // BWORK is not referenced for non-ordered Schur. |
|
11570
57632dea2446
attempt better backward compatibility for Array constructors
John W. Eaton <jwe@octave.org>
parents:
11523
diff
changeset
|
131 octave_idx_type ntmp = (ord_char == 'N' || ord_char == 'n') ? 0 : n; |
|
57632dea2446
attempt better backward compatibility for Array constructors
John W. Eaton <jwe@octave.org>
parents:
11523
diff
changeset
|
132 Array<octave_idx_type> bwork (dim_vector (ntmp, 1)); |
| 5275 | 133 octave_idx_type *pbwork = bwork.fortran_vec (); |
| 457 | 134 |
| 4552 | 135 F77_XFCN (zgeesx, ZGEESX, (F77_CONST_CHAR_ARG2 (&jobvs, 1), |
|
10314
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
136 F77_CONST_CHAR_ARG2 (&sort, 1), |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
137 selector, |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
138 F77_CONST_CHAR_ARG2 (&sense, 1), |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
139 n, s, n, sdim, pw, q, n, rconde, rcondv, |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
140 pwork, lwork, prwork, pbwork, info |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
141 F77_CHAR_ARG_LEN (1) |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
142 F77_CHAR_ARG_LEN (1) |
|
07ebe522dac2
untabify liboctave C++ sources
John W. Eaton <jwe@octave.org>
parents:
10158
diff
changeset
|
143 F77_CHAR_ARG_LEN (1))); |
| 457 | 144 |
| 145 return info; | |
| 146 } | |
| 10822 | 147 |
|
11498
367bfee35ba0
data member initialization fixes
John W. Eaton <jwe@octave.org>
parents:
11495
diff
changeset
|
148 ComplexSCHUR::ComplexSCHUR (const ComplexMatrix& s, const ComplexMatrix& u) |
|
367bfee35ba0
data member initialization fixes
John W. Eaton <jwe@octave.org>
parents:
11495
diff
changeset
|
149 : schur_mat (s), unitary_mat (u), selector (0) |
| 10822 | 150 { |
| 151 octave_idx_type n = s.rows (); | |
| 152 if (s.columns () != n || u.rows () != n || u.columns () != n) | |
| 153 (*current_liboctave_error_handler) | |
| 154 ("schur: inconsistent matrix dimensions"); | |
| 155 } | |
| 156 | |
| 157 ComplexSCHUR::ComplexSCHUR (const SCHUR& s) | |
|
11498
367bfee35ba0
data member initialization fixes
John W. Eaton <jwe@octave.org>
parents:
11495
diff
changeset
|
158 : schur_mat (s.schur_matrix ()), unitary_mat (s.unitary_matrix ()), |
|
367bfee35ba0
data member initialization fixes
John W. Eaton <jwe@octave.org>
parents:
11495
diff
changeset
|
159 selector (0) |
| 10822 | 160 { |
| 161 octave_idx_type n = schur_mat.rows (); | |
| 162 if (n > 0) | |
| 163 { | |
| 164 OCTAVE_LOCAL_BUFFER (double, c, n-1); | |
| 165 OCTAVE_LOCAL_BUFFER (double, sx, n-1); | |
| 166 | |
| 167 F77_XFCN (zrsf2csf, ZRSF2CSF, (n, schur_mat.fortran_vec (), | |
| 168 unitary_mat.fortran_vec (), c, sx)); | |
| 169 } | |
| 170 } |