Skip to content
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

Retun data incorrect for attach in Phash::FFI #71

Closed
lbe opened this issue Feb 24, 2017 · 2 comments
Closed

Retun data incorrect for attach in Phash::FFI #71

lbe opened this issue Feb 24, 2017 · 2 comments

Comments

@lbe
Copy link

lbe commented Feb 24, 2017

I have opened an rt.cpan tickett on Phash::FFI documenting a problem with what appears to be a truncation of a uint64 result to a uint32. While I am experiencing problems with Phash::FFI, I believe the problem is more likely with FFI::Platypus since that is where the heavy lifting is done. There is very little additional code in Phash::FFI and what is there passes my test. Phash::FFI is largely a configuration where 99% of the work is done here.

Please review the documentation in the linked ticket and advise if you would like me to try/test anything

Thanks,

lbe

https://rt.cpan.org/Ticket/Display.html?id=120345&results=a706a680eb7c7794fbd1ded3152355d0

@plicease
Copy link
Member

I can't test this right now because I am in the middle of nowhere Tasmania, but I think the arguments need to be uint64 not the return value.

@lbe
Copy link
Author

lbe commented Feb 26, 2017

Sir, you are correct!! I thought I tested this scenario; however, I must have either tested it wrong or misinterpreted the results in my sleep deprived state :) I will update the the Phash::FFI ticket accordingly.

Thank You!

lbe

@lbe lbe closed this as completed Feb 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants