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

Msvc fix #1

Merged
merged 2 commits into from
Oct 31, 2013
Merged

Msvc fix #1

merged 2 commits into from
Oct 31, 2013

Conversation

nu774
Copy link
Contributor

@nu774 nu774 commented Oct 31, 2013

commit a91457c
Author: nu774 honeycomb77@gmail.com
Date: Sat Oct 26 11:25:23 2013 +0900

fix unnecessary UTF-8 ligature of "fi" to ascii to shut up MSVC warnings

commit 3af068c
Author: nu774 honeycomb77@gmail.com
Date: Sat Oct 26 10:06:00 2013 +0900

change inttypes.h -> stdint.h, fix #ifdef condition for MSVC

These C99 integer types are declared in stdint.h, so inttypes.h is
not required.
Recent MSVC provides stdint.h with different typedefs from wavpack.h,
resulting in build issues on third party software that includes
both stdint.h and wavpack.h.

_MSC_VER <  1600:   no stdint.h
_MSC_VER <  1800:   provides stdint.h, no inttypes.h
_MSC_VER >= 1800:   provides both stdint.h and inttypes.h

These C99 integer types are declared in stdint.h, so inttypes.h is
not required.
Recent MSVC provides stdint.h with different typedefs from wavpack.h,
resulting in build issues on third party software that includes
both stdint.h and wavpack.h.

_MSC_VER <  1600:   no stdint.h
_MSC_VER <  1800:   provides stdint.h, no inttypes.h
_MSC_VER >= 1800:   provides both stdint.h and inttypes.h
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

Successfully merging this pull request may close these issues.

2 participants