commit 762ec445e63a95c1545b18b28d528f8ce38a9afd
author ocornut <omarcornut@gmail.com> Mon Aug 28 11:24:29 2023 +0200
committer ocornut <omarcornut@gmail.com> Mon Aug 28 11:25:50 2023 +0200
tree f1613c71f2f429fd173166d31dd8fefcf4c4ffb2
parent 475453668d7da1662bf90266c5f0df0259705987

Docking: Fixed bad bug, dock node host wouldn't merge background. (#6753, #6716)

Fixes f422e78. Didn't crash because g.Windows[] is always larger than g.WindowsFocusOrder[].

imgui.cpp

diff --git a/imgui.cpp b/imgui.cpp
index 7bce4e4..059d8c3 100644
--- a/imgui.cpp
+++ b/imgui.cpp
@@ -5341,20 +5341,16 @@
     CallContextHooks(&g, ImGuiContextHookType_RenderPre);
 
     // Add background ImDrawList (for each active viewport)
-    for (int n = 0; n != g.Viewports.Size; n++)
+    for (ImGuiViewportP* viewport : g.Viewports)
     {
-        ImGuiViewportP* viewport = g.Viewports[n];
         InitViewportDrawData(viewport);
         if (viewport->BgFgDrawLists[0] != NULL)
             AddDrawListToDrawDataEx(&viewport->DrawDataP, viewport->DrawDataBuilder.Layers[0], GetBackgroundDrawList(viewport));
     }
 
-    for (int n = 0; n != g.WindowsFocusOrder.Size; n++)
-    {
-        ImGuiWindow* window = g.Windows[n];
+    for (ImGuiWindow* window : g.WindowsFocusOrder)
         if (window->Flags & ImGuiWindowFlags_DockNodeHost)
             window->DrawList->ChannelsMerge();
-    }
 
     // Draw modal/window whitening backgrounds
     RenderDimmedBackgrounds();