New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix memory, initialization and other coverity issues in Framework (especially Load* classes) #390
Fix memory, initialization and other coverity issues in Framework (especially Load* classes) #390
Conversation
@@ -115,16 +115,16 @@ class DLLExport LoadMask : public API::Algorithm { | |||
std::vector<int> &rangeendvec); | |||
|
|||
/// Mask Workspace | |||
DataObjects::MaskWorkspace_sptr mMaskWS; | |||
DataObjects::MaskWorkspace_sptr m_MaskWS; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect when you renamed this, you didn't mean to capitalise the M
:)
Same goes for m_DefaultToUse
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good that you caught this. I'm fixing it...
Other than that it looks good. Nice big clean up for Coverity 👍 |
…_issues_framework_load_classes Fix memory, initialization and other coverity issues in Framework (especially Load* classes)
This fixes ~30 issues listed in ticket #11328.
I tried to get rid of most if not all remaining issues in the Load* classes, excluding security issues and a handful of unclear ones.
Tester: code review, check that all tests are ok, which cover quite decently most of the changes included here.