-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[flann] Fix pop_t is not defined under non-gunc platform. #32468
Conversation
No, the correct fix is to remove its usage: |
OK, I'll fix it. |
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.
Please follow up with upstream to ensure the change is correct.
ports/flann/portfile.cmake
Outdated
@@ -9,6 +9,7 @@ vcpkg_from_github( | |||
PATCHES | |||
fix-build-error.patch | |||
fix-dependency-hdf5.patch | |||
fix_undefined_pot_t.patch #https://github.com/flann-lib/flann/pull/405 |
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.
The changes in this patch are not the changes in that PR.
HammingLUT lut; | ||
result = lut(reinterpret_cast<const unsigned char*> (a), | ||
- reinterpret_cast<const unsigned char*> (b), size * sizeof(pop_t)); | ||
+ reinterpret_cast<const unsigned char*> (b), size); |
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 seems broken as the linked PR suggests that pop_t
should be either unsigned long
or unsigned long long
, which means size will be too small with this change.
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.
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 see, so the patch is correct because the original code was bogus, and the comment is wrong.
Thanks! |
Fixes #32448
Add fix_undefined_pop_t.patch.(https://github.com/opencv/opencv/pull/13270)
Usage test pass with following triplets:
SHA512s are updated for each updated downloadThe "supports" clause reflects platforms that may be fixed by this new versionAny fixed CI baseline entries are removed from that file.Any patches that are no longer applied are deleted from the port's directory../vcpkg x-add-version --all
and committing the result.