On Sat Sep 16 17:58:03 2023 +0000, Alex Henrie wrote:
Sorry I missed your message. In setupapi and urlmon, _recalloc was accepted as a replacement for HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, ...). I got the impression that it was OK to use _recalloc, just that it was not worth replacing existing code that uses realloc and memset with _recalloc. I have updated these patches to use realloc and memset instead as you requested. (Actually, it turned out that in 3 of the 6 uses here, memset was unnecessary because the memory was already being initialized.)
Surely _recalloc() works, but my point is that we don't need it, and more importantly don't need to limit wine modules to only work with ucrt.