Feature/tidy up linsys
Issue/feature addressed
This is just a minor tidy-up of the LinSys class and subclasses. Here is a summary of the changes:
- Remove unused variable
min_resid
inNekLinSysIterCG.cpp
. - Remove redundant counter variable
k
inNekLinSysIterCG.cpp
. - Remove unnecessary
boost::ignore_unused
function call in several files. - Change
Array<OneD, NekDouble> vExchange(1, 0.0)
toNekDouble vExchange(0.0)
inNekLinSysIter.cpp
. - Remove unnecessary lvalue assignment in NekVector constructor call in
NekLinSysIterCG.cpp
. - Fix some documentation errors/typos in
library/LibUtilities/BasicUtils/Vmath.hpp
- Fix some documentation errors/typos in
library/LibUtilities/BasicUtils/Vmath.cpp
- Fix some documentation errors/typos in
library/LibUtilities/BasicUtils/VmathArray.hpp
- Fix some documentation errors/typos in
library/LibUtilities/BasicUtils/VmathSIMD.hpp
Proposed solution
Implementation
Tests
Suggested reviewers
Please suggest any people who would be appropriate to review your code.
Notes
Please add any other information that could be useful for reviewers.
Checklist
[ ] Functions and classes, or changes to them, are documented.[ ] User guide/documentation is updated.-
Changelog is updated. [ ] Suitable tests added for new functionality.-
Contributed code is correctly formatted. (See the contributing guidelines). [ ] License added to any new files.-
No extraneous files have been added (e.g. compiler output or test data files).
Edited by Jacques Xing