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
cosmoelogic wants to merge 1 commit into4coder-archive:masterfrom
cosmoelogic:patch-1
Open

Silence MSVC warning C4389#5
cosmoelogic wants to merge 1 commit into4coder-archive:masterfrom
cosmoelogic:patch-1

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?

@cristeigabriela
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