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

Preventing C4244 warnings in GetBounds() etc. #830

Closed
wants to merge 1 commit into from
Closed

Preventing C4244 warnings in GetBounds() etc. #830

wants to merge 1 commit into from

Conversation

tkhisak
Copy link

@tkhisak tkhisak commented May 7, 2024

When using Clipper2Lib::ScalePaths() for integer type Paths64, a C4244 warning occurs in the call to GetBounds<double, T2>(paths) (around line 530 of clipper.core.h) . (Using Visual Studio/C++.)
I would like to add a static cast to avoid this warning.

@AngusJohnson
Copy link
Owner

Thank you for the feedback.
However, I guess for completeness, static casts should also be applied in lines 458-461 too.
These I've done too and will upload momentarily.

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