ArbitraryDomain::globalToLocalQuaternion_m also defined in its parent class 'IrregularDomain
From issue #62. [src/Solvers/ArbitraryDomain.h:84] -> [src/Solvers/IrregularDomain.h:123]: (warning) The class 'ArbitraryDomain' defines member variable with name 'globalToLocalQuaternion_m' also defined in its parent class 'IrregularDomain'.
Shadowing members is very error-prone and was probably not intended.
A quick look at the code seems to show that the derived ArbitraryDomain uses its globalToLocalQuaternion_m as a read-only copy of its parent member. So ArbitraryDomain::globalToLocalQuaternion_m can be safely substituted for a local variable since there is a public getter method (and even the actual parent member is directly accessible (protected)).