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
Topic/ebsdlib memory crashing fixes #859
Topic/ebsdlib memory crashing fixes #859
Conversation
The last set of changes introduced memory leaks and crashes inside of the various CTF readers. This was caused by too much optimization and creating too much consistency to make the CTFReader look like the ANGReader even though the two readers are implemented in vastly different ways. + Removed test files that are no longer relevant + Removed other debugging statements from the H5EspritReader + Applied clang-tidy suggestions where possible to all of EBSDLib Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
33a699a
to
0a7984e
Compare
Source/EbsdLib/HKL/CtfHeaderEntry.h
Outdated
@@ -1,3 +1,7 @@ | |||
#include <utility> | |||
|
|||
#include <utility> |
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.
Only one #include <utility>
required
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.
Fix the double include, and this is good.
Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
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.
Looks good.
…uctor Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
* Fix memory leaks and crashing issues in EBSDLib for Oxford readers The last set of changes introduced memory leaks and crashes inside of the various CTF readers. This was caused by too much optimization and creating too much consistency to make the CTFReader look like the ANGReader even though the two readers are implemented in vastly different ways. + Removed test files that are no longer relevant + Removed other debugging statements from the H5EspritReader + Applied clang-tidy suggestions where possible to all of EBSDLib + Fix warnings about classes with virtual methods but non-virtual destructor Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
No description provided.