Skip to content
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.

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

size_t set to signed values #37

Closed
pcarbo opened this issue May 9, 2017 · 1 comment
Closed

size_t set to signed values #37

pcarbo opened this issue May 9, 2017 · 1 comment

Comments

@pcarbo
Copy link
Collaborator

pcarbo commented May 9, 2017

@xiangzhou According to the C++ standard, size_t is an unsigned integer type, but you are assigning (constant) negative values to variables of this type, which seems to be generating warnings in some cases; e.g., in io.cpp:

SNPINFO sInfo={"-9", rs, -9, -9, minor, major, -9, -9, (long int) -9};

So you should either replace these with unsigned values, or change the definition of SNPINFO.

@pjotrp
Copy link
Member

pjotrp commented Nov 23, 2017

I removed all compiler warnings in the latest build. Added only one cast in mvlmm.cpp.

@pjotrp pjotrp closed this as completed Nov 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants