-
Notifications
You must be signed in to change notification settings - Fork 167
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
replaced custom min/max with std::max/std::min #5490
Conversation
isis/src/base/objs/Cube/Cube.h
Outdated
@@ -451,7 +451,6 @@ namespace Isis { | |||
|
|||
//! If allocated, converts from physical on-disk band # to virtual band # | |||
QList<int> *m_virtualBandList; | |||
}; |
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.
This broke building ISIS
Like Adam mentioned a few months ago. Build is broken:
|
DefaultCube.FunctionalTestNoprojExpand and isis_unit_test_NaifDskShape are failing, I'll take a look at these |
isis_unit_test_NaifDskShape is failing on dev too, seems related to the egos segfault issue? DefaultCube.FunctionalTestNoprojExpand passes locally (probably a Mac thing) but also fails on dev. So I think tests are good rn. |
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.
Changes looked fine, now that the build is working this should be good to go
Description
Replacing many custom MIN/MAX functions with std library versions.
Related Issue
no related issue. Needed to make these changes as part of adding GDAL support for cubes. Got a bunch of errors after adding the GDAL dependency.
How Has This Been Validated?
Still need to run tests locally.
Types of changes
Checklist:
Licensing
This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words: