May 29, 2026
5:05 p.m.
On Fri May 29 16:50:29 2026 +0000, Alex Henrie wrote:
I thought about doing that, then realized that if we wait to allocate `data` until it is actually needed then `free(data)` only needs to be called in the last error case and not in every error case. In my opinion that makes the code more clear. But either way, initializing `data` to `NULL` is redundant because `data = malloc(sizeof(*data))` is always run before `free(data)` is called. Ah, I see. That's fair as well.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10931#note_141727