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.

Sign up for GitHub

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

Jump to bottom

check overflow on getting field id #3075

Open

jp0317 wants to merge 1 commit into apache:master from jp0317:jp0317-overflow

jp0317 commented Dec 12, 2024

Add overflow checking when getting field id based on field delta.


          check overflow on reading field id

67bf818

fishy added the rust label

Member

Jens-G commented Feb 28, 2025

Sth is not right here

 error: use of `ok_or` followed by a function call

Author

jp0317 commented Feb 28, 2025

yeah, it's should be fine to use ok_or there. this seems to be a false alarm (found a similar issue here)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust