checkMatrix return type refinement
Open
stale
- Dominant language
- C++
- Stars
- 403
- Forks
- 154
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 82
Description
I saw repetitive line `CHECKED_ELSE(check.result) { FAIL(check.result_message); }` in https://github.com/QMCPACK/qmcpack/pull/4732/files
The code can be simplified by returning bool and throw error in checkMatrix instead of `{bool, string}`
Contributor guide
Assessment
This issue has not been assessed yet.