Backends: SDL3: following SDL3 reverting their change, result of SDL_GetDisplays() must be freed. (#7809)

Reverts 271910e
This commit is contained in:
ocornut 2024-09-03 19:13:45 +02:00
parent 226e0d579d
commit 6af9ac29ec

View File

@ -818,7 +818,7 @@ static void ImGui_ImplSDL3_UpdateMonitors()
bd->WantUpdateMonitors = false; bd->WantUpdateMonitors = false;
int display_count; int display_count;
const SDL_DisplayID* displays = SDL_GetDisplays(&display_count); SDL_DisplayID* displays = SDL_GetDisplays(&display_count);
for (int n = 0; n < display_count; n++) for (int n = 0; n < display_count; n++)
{ {
// Warning: the validity of monitor DPI information on Windows depends on the application DPI awareness settings, which generally needs to be set in the manifest or at runtime. // Warning: the validity of monitor DPI information on Windows depends on the application DPI awareness settings, which generally needs to be set in the manifest or at runtime.
@ -839,6 +839,7 @@ static void ImGui_ImplSDL3_UpdateMonitors()
continue; // Some accessibility applications are declaring virtual monitors with a DPI of 0, see #7902. continue; // Some accessibility applications are declaring virtual monitors with a DPI of 0, see #7902.
platform_io.Monitors.push_back(monitor); platform_io.Monitors.push_back(monitor);
} }
SDL_free(displays);
} }
void ImGui_ImplSDL3_NewFrame() void ImGui_ImplSDL3_NewFrame()