Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.

Silence MSVC warning C4389 #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cosmoelogic
Copy link

"error C2220: the following warning is treated as an error
warning C4389: '==': signed/unsigned mismatch"
This prevents it from compiling

@RealAstolfo
Copy link

A better solution would likely be to actually fix the fact that it is comparing an unsigned with a signed. Simply silencing warnings where they pop will eventually lead to UB down the line. Where is this issue showing up? File / line?

@cristeigabriel
Copy link

A better solution would likely be to actually fix the fact that it is comparing an unsigned with a signed. Simply silencing warnings where they pop will eventually lead to UB down the line. Where is this issue showing up? File / line?

The issue is related to the macro itself.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants