player: remove the last instances of polling

Mouse cursor handling, --heartbeat-cmd, and OSD messages basically
relied on polling. For this reason, the playloop always used a small
timeout (not more than 500ms).

Fix these cases, and raise the timeout to 100 seconds. There is no
reason behind this number; for this specific purpose it's as close to
infinity as any other number.

On MS Windows, or if vo_sdl is used, the timeout remains very small.
In these cases the GUI code doesn't do proper event handling in the
first place, and fixing it requires much more effort.

getch2_poll() still does polling, because as far as I'm aware no event-
based way to detect this state change exists.

1 files changed, 9 insertions, 7 deletions

diff --git a/player/osd.c b/player/osd.c
index 939d62fd98..41d0b8d160 100644
--- a/player/osd.c
+++ b/player/osd.c
@@ -314,19 +314,21 @@ static mp_osd_msg_t *get_osd_msg(struct MPContext *mpctx)
     mp_osd_msg_t *msg = mpctx->osd_msg_stack;
     if (msg) {
         if (!msg->started || msg->time > diff) {
+            // display it
             if (msg->started)
                 msg->time -= diff;
             else
                 msg->started = 1;
-            // display it
-            return msg;
+        } else {
+            // kill the message
+            talloc_free(msg);
+            msg = NULL;
+            mpctx->osd_msg_stack = NULL;
         }
-        // kill the message
-        talloc_free(msg);
-        mpctx->osd_msg_stack = NULL;
     }
-    // Nothing found
-    return NULL;
+    if (msg)
+        mpctx->sleeptime = MPMIN(mpctx->sleeptime, msg->time);
+    return msg;
 }
 
 // type: mp_osd_font_codepoints, ASCII, or OSD_BAR_*