Skip to content

Conversation

@beskidlabs
Copy link

Using the lib in embedded environments, causes issues with compilation using ARM embedded toolchains. The call to round() is being resolved as a call to lround() and hence breaks the build, as std::max() expects two arguments of type double. Making it a explicit call to std::round() fixes the issue.

Using the lib in embedded environments, causes issues with compilation using
ARM embedded toolchains. The call to round() is being resolved as a call to
lround() and hence breaks the build, as std::max() expects two arguments of
type double. Making it a explicit call to std::round() fixes the issue.
@google-cla
Copy link

google-cla bot commented Dec 2, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@wowo68
Copy link

wowo68 commented Dec 2, 2025 via email

@beskidlabs
Copy link
Author

Well, I don't want to go through all that formalities (I don't have nor want to have a google account) just to submit such a miniscule change, perhaps it's better if I submit it via issue tracking system..

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants