| Summary: | Unresolved external symbol memset referenced in function SDL_vsnprintf_REAL | ||
|---|---|---|---|
| Product: | SDL | Reporter: | Tom Hulton-Harrop <tom.hultonharrop> |
| Component: | build | Assignee: | Sam Lantinga <slouken> |
| Status: | RESOLVED DUPLICATE | QA Contact: | Sam Lantinga <slouken> |
| Severity: | blocker | ||
| Priority: | P2 | CC: | sezeroz |
| Version: | HG 2.1 | ||
| Hardware: | x86_64 | ||
| OS: | Windows 10 | ||
|
Description
Tom Hulton-Harrop
2020-10-31 21:42:48 UTC
MSCV Version - MSVC 19.27.29111.0 (In reply to Tom Hulton-Harrop from comment #0) > Fix: Add vcruntime here: Wrong thing to do: It is supposed to build and link w/o any MSC runtime. Maybe we should try a version of the solution suggested in the second link you referenced, i.e. something like #pragma function(memset) void* __cdecl memset(void* dest, int c, size_t count) { SDL_memset(dest, c, count); } Okay understood, yes the second point might be a better option, the first just seemed easier/faster if it could be accepted. Not sure if the other change is potentially risky or not, hopefully there's a unit test to cover it in case :) I wound up grabbing the latest SDL build (SDL-2.0.13-14202.zip) and this error no longer exists so this bug can most likely be closed. Sorry for any inconvenience! (In reply to Tom Hulton-Harrop from comment #4) > I wound up grabbing the latest SDL build (SDL-2.0.13-14202.zip) and this > error no longer exists so this bug can most likely be closed. > > Sorry for any inconvenience! Huh. Missed that this is actually a duplicate of bug #5112, and is fixed by https://hg.libsdl.org/SDL/rev/d5fe4ad4d29c Closing. *** This bug has been marked as a duplicate of bug 5112 *** Okay great to hear, thanks for the update. |