Skip to content

stdint.h Is Not Included #537

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

Closed
yiqiju opened this issue Sep 27, 2016 · 4 comments
Closed

stdint.h Is Not Included #537

yiqiju opened this issue Sep 27, 2016 · 4 comments

Comments

@yiqiju
Copy link

yiqiju commented Sep 27, 2016

On 1.7.6, looks like config.h trying to use int64_t but does not include stdint.h on Linux.

Details:
include/json/config.h:155 'int64_t' does not name a type

@yiqiju
Copy link
Author

yiqiju commented Sep 28, 2016

Pull request #538 is submitted

@cdunn2001
Copy link
Contributor

I think we would use #include <cstdint> since we are C++11 compatible. That might break an old VisualStudio version, but that's probably ok. See #509.

@yiqiju
Copy link
Author

yiqiju commented Sep 28, 2016

I suggest to keep it as-is since it builds on both old and newer compilers.

On Tue, Sep 27, 2016, 18:04 Christopher Dunn [email protected]
wrote:

I think we would use #include since we are C++11 compatible.
That might break an old VisualStudio version, but that's probably ok. See
#509 #509.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#537 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABoS2R9TPv-5uAMdBLWWavQwEbDr4g_bks5qub0WgaJpZM4KIRe1
.

@cdunn2001
Copy link
Contributor

Well, we have stdlib.h and stddef.h, so ok. Merging...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants