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.

Sign up for GitHub

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

Jump to bottom

restore mingw #16

Closed
rurban opened this issue Sep 4, 2017 · 1 comment
Closed

restore mingw #16

rurban opened this issue Sep 4, 2017 · 1 comment
Assignees
@rurban
Labels
bug
Milestone
3.0

Comments

@rurban
Copy link
Owner

rurban commented Sep 4, 2017 •
edited

  • detect mingw and mingw cross (for testing with wine).
  • skip conflicting declarations with their non-C11 MINGW_HAS_SECURE_API:
    strtok_s, vsnprintf_s, wcstok_s
  • note their extensions, esp. their _l locale versions.
  • add dllimport/dllexport decl and cleanup the header mess in the sources.
./configure --enable-unsafe --host=x86_64-w64-mingw32
wchar/vswprintf_s.c:160: warning: passing arg 2 of `vswprintf' makes pointer from integer without a cast
wchar/vswprintf_s.c:160: warning: passing arg 3 of `vswprintf' from incompatible pointer type
wchar/vswprintf_s.c:160: error: too many arguments to function `vswprintf'
@rurban rurban self-assigned this Sep 4, 2017
@rurban rurban added the bug label Sep 4, 2017
@rurban rurban added this to the 3.0 milestone Sep 4, 2017
rurban added a commit that referenced this issue Sep 4, 2017
@rurban
mingw cross and compat
3e412d4 
support cross-compilation for mingw (w64 and mingw32)
check for conflicts with their non-C11 MINGW_HAS_SECURE_API:
strtok_s, vsnprintf_s, wcstok_s

add EXPORT and EXTERN helpers for dllexport/dllimport
all sources now only need #include "safeclib_private.h", nothing
else.

fixed a bug in the un,likely macros, found only by the stricter mingw64 gcc.

See #16
rurban added a commit that referenced this issue Sep 4, 2017
@rurban
mingw cross and compat
ddcf4e9 
support cross-compilation for mingw (w64 and mingw32)
check for conflicts with their non-C11 MINGW_HAS_SECURE_API:
strtok_s, vsnprintf_s, wcstok_s

add EXPORT and EXTERN helpers for dllexport/dllimport
all sources now only need #include "safeclib_private.h", nothing
else.

fixed a bug in the un,likely macros, found only by the stricter mingw64 gcc.

check for langinfo.h (not on mingw)

See #16
rurban added a commit that referenced this issue Sep 4, 2017
@rurban
mingw cross and compat
f9df87e 
support cross-compilation for mingw (w64 and mingw32)
check for conflicts with their non-C11 MINGW_HAS_SECURE_API:
strtok_s, vsnprintf_s, wcstok_s

add EXPORT and EXTERN helpers for dllexport/dllimport
all sources now only need #include "safeclib_private.h", nothing
else.

fixed a bug in the un,likely macros, found only by the stricter mingw64 gcc.

check for langinfo.h (not on mingw)

See #16
@rurban
Copy link
Owner Author

rurban commented Sep 5, 2017

Done with de091c4

@rurban rurban closed this as completed Sep 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rurban rurban

Labels
bug
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
1 participant
@rurban