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.

Sign up for GitHub

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

Jump to bottom

DSPAnalyzer: Fix two clang warnings about sign mismatched types #4580

Merged

degasus merged 1 commit into dolphin-emu:master from lioncash:warn

Dec 29, 2016

Member

lioncash commented Dec 29, 2016

[10:05] <flacs> Lioncash: you introduced two clang warnings in a8459fc
[10:06] <flacs> by replacing a #define with a size_t variable you made it unsigned, but it's being compared to a signed value
[10:07] <flacs> (NUM_IDLE_SIGS and MAX_IDLE_SIG_SIZE)


          DSPAnalyzer: Fix two clang warnings about sign mismatched types

efbf261

degasus merged commit f499145 into dolphin-emu:master

lioncash deleted the warn branch

December 29, 2016 23:08

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