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

fix UWP build error #21

Merged
merged 2 commits into from
May 29, 2020
Merged

fix UWP build error #21

merged 2 commits into from
May 29, 2020

Conversation

myd7349
Copy link
Contributor

@myd7349 myd7349 commented May 29, 2020

Hi! I was trying to add wavelib into vcpkg recently: microsoft/vcpkg#11611. And the CI system of vcpkg reports build failures on x64-uwp and arm-uwp. The build logs can be downloaded from here: https://dev.azure.com/vcpkg/public/_build/results?buildId=37972&view=artifacts&type=publishedArtifacts

This patch tries to:

  • fix the following UWP build errors caught by vcpkg's CI
    src\wavelib.c(3673): error C4703: potentially uninitialized local pointer variable 'out' used
    src\wavelib.c(3499): error C4703: potentially uninitialized local pointer variable 'wavecoeff' used
  • move the definition of out variable to support old C89 compilers;
    That way, I can use wavelib with VS2010.

- fix the following UWP build errors caught by vcpkg's CI
  src\wavelib.c(3673): error C4703: potentially uninitialized local pointer variable 'out' used
  src\wavelib.c(3499): error C4703: potentially uninitialized local pointer variable 'wavecoeff' used
- move the definition of `out` variable to support old C89 compilers;
@myd7349
Copy link
Contributor Author

myd7349 commented May 29, 2020

This patch also removes redundant target_link_libraries(... m) in test/CMakeLists.txt which should have been removed in the previous PR.

@myd7349 myd7349 marked this pull request as ready for review May 29, 2020 04:10
@rafat rafat merged commit c7c9ba8 into rafat:master May 29, 2020
@myd7349 myd7349 deleted the fix-uwp-build branch May 29, 2020 08:46
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