Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Use PRId32 in c99 environment. #482

wants to merge 1 commit into


None yet
2 participants

monaka commented Sep 29, 2012

This issue is discussed before. It is the second try.
In case sizeof(uint32_t) != sizeof(unsigned int), we got warnings at cdump.c.
Using PRId32 macro is the best way to avoid it.
But terrible VC++ doesn't have PRId32 that is supported since C99.

This patch is to define PRId32 as "x" if the complier doesn't support C99.

@monaka monaka referenced this pull request Jan 24, 2013


Fixed MRB_INT64 warning #766


masuidrive commented Jan 29, 2013

I think it's fixed by #766 .
Could you close it?


monaka commented Jan 29, 2013

Ok. I close this. Thanks.

@monaka monaka closed this Jan 30, 2013

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