On Sat Sep 16 17:57:34 2023 +0000, Alex Henrie wrote:
changed this line in [version 3 of the diff](/wine/wine/-/merge_requests/3617/diffs?diff_id=69818&start_sha=96ffbcd6dfa72980c2fe29a0868ee7e4b579cfc1#fb019c552bf00affb90208dc4b91a0d82578f9e9_217_219)
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.)