comparison liboctave/numeric/svd.h @ 22228:4afe3705ea75

svd: do not shadow member variables with local variables. * liboctave/numeric/svd.cc, liboctave/numeric/svd.h: there was no issue with current code because member variables never change after construction. Still, it's not good practice and the compilers will warn about it.
author Carnë Draug <carandraug@octave.org>
date Tue, 09 Aug 2016 00:21:36 +0100
parents 469c817eb256
children e43199618777
comparison
equal deleted inserted replaced
22227:ab139f0733b9 22228:4afe3705ea75
48 GESVD, 48 GESVD,
49 GESDD 49 GESDD
50 }; 50 };
51 51
52 svd (void) 52 svd (void)
53 : type (), driver (), left_sm (), sigma (), right_sm () 53 : m_type (), m_driver (), left_sm (), sigma (), right_sm ()
54 { } 54 { }
55 55
56 svd (const T& a, svd::Type type = svd::Type::std, 56 svd (const T& a, svd::Type type = svd::Type::std,
57 svd::Driver driver = svd::Driver::GESVD); 57 svd::Driver driver = svd::Driver::GESVD);
58 58
59 svd (const svd& a) 59 svd (const svd& a)
60 : type (a.type), driver (a.driver), left_sm (a.left_sm), sigma (a.sigma), 60 : m_type (a.m_type), m_driver (a.m_driver), left_sm (a.left_sm),
61 right_sm (a.right_sm) 61 sigma (a.sigma), right_sm (a.right_sm)
62 { } 62 { }
63 63
64 svd& operator = (const svd& a) 64 svd& operator = (const svd& a)
65 { 65 {
66 if (this != &a) 66 if (this != &a)
67 { 67 {
68 type = a.type; 68 m_type = a.m_type;
69 left_sm = a.left_sm; 69 left_sm = a.left_sm;
70 sigma = a.sigma; 70 sigma = a.sigma;
71 right_sm = a.right_sm; 71 right_sm = a.right_sm;
72 driver = a.driver; 72 m_driver = a.m_driver;
73 } 73 }
74 74
75 return *this; 75 return *this;
76 } 76 }
77 77
86 private: 86 private:
87 87
88 typedef typename T::element_type P; 88 typedef typename T::element_type P;
89 typedef typename DM_T::element_type DM_P; 89 typedef typename DM_T::element_type DM_P;
90 90
91 svd::Type type; 91 svd::Type m_type;
92 svd::Driver driver; 92 svd::Driver m_driver;
93 93
94 T left_sm; 94 T left_sm;
95 DM_T sigma; 95 DM_T sigma;
96 T right_sm; 96 T right_sm;
97 97