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 for Windows build using MSVC #12

Closed
wants to merge 1 commit into from
Closed

Fix for Windows build using MSVC #12

wants to merge 1 commit into from

Conversation

jasny
Copy link

@jasny jasny commented Jun 25, 2019

Variable length arrays are not supported by MSVC. Fall back to malloc.

Based on the fix by holland01 #7, but

  • No goto. Instead do a free for each return. (Ignored for GCC build.)
  • No output when out of mem, just return false.
  • No 'style' changes.

@jasny
Copy link
Author

jasny commented Jul 4, 2019

@luke-jr 🏓

@ikeedge

This comment was marked as spam.

hernanrz added a commit to hernanrz/libaergo that referenced this pull request Dec 17, 2022
hernanrz added a commit to hernanrz/libaergo that referenced this pull request Dec 18, 2022
based on this patch bitcoin/libbase58#12

make aergo.h compile on MSVC

add dll entrypoint for windows

copied from https://github.com/aergoio/secp256k1-vrf/blob/master/.github/workflows/build.yml#L34

windows compilation
@jasny jasny closed this by deleting the head repository May 29, 2023
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