SDL hangs all event processing on window move, causing ImGui to crash about magnum-integration HOT 6 CLOSED

Interesting. What's the platform / compiler you're on? Is it possible that the app is running at more than 60 FPS (indicating broken VSync, probably) which is causing the delta time to be very close to zero?

from magnum-integration.

Windows / MinGW 7.3.0
VSync on

from magnum-integration.

If you try to drag the window

Ah! Now I see -- you mean dragging the application window, not the ImGui windows, right? This could be because, IIRC, on Windows each window drag triggers a repaint for some reason, causing drawEvent() to be called much more than @ 60 FPS.

I'll need to investigate why this frequent redraw is done, if it's needed at all and how ImGui itself handles this in its own examples. Thanks for the report, I'll ping you back once I have something.

from magnum-integration.

Yes, I mean application window. Ok. Thanks.

from magnum-integration.

Unfortunately I don't really have a solution for this. SDL has a Window-specific bug/misfeature, reported back in 2014 and not fixed since, where it hangs all event processing for the time where you drag or resize the window. After that, it flushes the event queue, causing repaint events and everything else to happen immediately after each other. That ImGui crashes is only a consequence, it's very likely to cause serious misbehavior elsewhere as well -- I hit this most recently with mosra/magnum#423 when trying to implement DPI scaling change events, there it delays the window resize to when you release the mouse again, not nice.

I am monitoring that bug, hoping something will happen in this decade at least.

Two suggestions -- either don't drag windows with ImGui in it, or switch to GlfwApplication, GLFW doesn't suffer from this problem at all.

from magnum-integration.

Worked around in e7dc17a basically the same way you suggested -- there's nothing happening about this at on the SDL side at all (the bug is milestoned for SDL 2.1, who knows when that happens), so let's do a fix here instead.

from magnum-integration.