bkacjios / m-overlay

A controller input display/overlay for Dolphin
MIT License
165 stars 44 forks source link

Application hangs when closing #169

Closed jjvliu closed 1 year ago

jjvliu commented 1 year ago

Describe the bug Opening then shortly closing M'Overlay 2.1.1 without even opening Slippi sometimes causes M'Overlay to hang. This happens to me roughly 150 out of 200 times with no clear pattern. When M'Overlay hangs, it can be properly closed by closing the debug popup window.

Desktop (please complete the following information):

To Reproduce Steps to reproduce the behavior:

  1. Open M'Overlay in debugging mode.
  2. (Optional) Interact with the UI by clicking around and/or pressing escape.
  3. Click the X to close M'Overlay. This will often cause it to stop responding.
  4. If M'Overlay stops responding, click the X on the debug console to properly close the application.

Screenshots image image

Debug console output

[DEBUG - 22:47:41] M'Overlay (2.1.1)
[DEBUG - 22:47:41] Love2D 11.4.0 - Mysterious Mysteries
[DEBUG - 22:47:41] Lua 5.1 (LuaJIT 2.1.0-beta3 - x64)
[DEBUG - 22:47:41] System: Windows
[DEBUG - 22:47:41] Renderer: OpenGL (3.3.14831 Core Profile Context 21.10.03.11 27.20.21003.11000)
[DEBUG - 22:47:41] Graphics: ATI Technologies Inc. (Radeon RX Vega M GH Graphics)

Additional context Possibly related to #164 but I thought I would record this very minimal working example. I also cannot reproduce the bug on the debug build 2.1.0-2-g0d95481 from this comment: https://github.com/bkacjios/m-overlay/issues/164#issuecomment-1461112812_

bkacjios commented 1 year ago

This is another thing I can't reproduce. I closed and reopened the application at least 50 times without a single crash. I do have a hunch that it's getting stuck shutting down the thread that handles updating the application for some reason.

bkacjios commented 1 year ago

I pushed out a new update to hopefully fix your crash on close issue.

jjvliu commented 1 year ago

Today I confirmed my issue persists on 2.1.1a. I also still saw #164 on 2.1.1 but have not tested it on 2.1.1a.