Skip to content

Defining NOMINMAX for windows to prevent std::{min,max} issues#1003

Open
COM8 wants to merge 2 commits intomasterfrom
fix/nominmax_windows
Open

Defining NOMINMAX for windows to prevent std::{min,max} issues#1003
COM8 wants to merge 2 commits intomasterfrom
fix/nominmax_windows

Conversation

@COM8
Copy link
Member

@COM8 COM8 commented Dec 28, 2023

Fixes #957.

Still in draft mode since it needs more testing.

@COM8 COM8 force-pushed the fix/nominmax_windows branch from 544c9fc to 0beb518 Compare December 28, 2023 14:59
@COM8 COM8 marked this pull request as ready for review January 20, 2024 18:22
@maztheman
Copy link

IMHO the best way to deal with this is to change std::min() to (std::min)()

@nlogozzo
Copy link
Contributor

+1

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

minwindef.h min and max macros conflicting with std::numeric_limits

3 participants