Reduced a sign-compare warning. #14

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

Open

verpeteren wants to merge 1 commit into nidium:master from verpeteren:sign-compare

Contributor

verpeteren commented Apr 14, 2015

The callback signature returns an int. The variable ret is not used for calculation.


          Reduced a sign-compare warning.

94258c1

Contributor

efyx commented Jan 31, 2017

Just tried this one and my compiler (clang3.9) does not throw any warning. However, we may still want to merge this one as it more correct.

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

Labels

None yet