Main: replace end_input_event() with an IdleEvent

src/AsyncUserInput.cxx

@@ -7,7 +7,6 @@
 #include "Command.hxx"
 #include "Bindings.hxx"
 #include "GlobalBindings.hxx"
-#include "ncmpc.hxx"
 #include "ui/Point.hxx"
 
 // TODO remove this kludge
@@ -39,7 +38,6 @@ AsyncUserInput::OnSocketReady(unsigned) noexcept
 			/* cancel the modal dialog on mouse click;
 			   though some day, we may allow modal dialogs
 			   to handle mouse events */
-			end_input_event();
 			return;
 		}
 
@@ -55,14 +53,11 @@ AsyncUserInput::OnSocketReady(unsigned) noexcept
 		if (!handler.OnMouse({event.x, event.y}, event.bstate))
 			return;
 
-		end_input_event();
-
 		return;
 	}
 #endif
 
 	if (handler.OnRawKey(key)) {
-		end_input_event();
 		return;
 	}
 
@@ -73,7 +68,6 @@ AsyncUserInput::OnSocketReady(unsigned) noexcept
 	if (!handler.OnCommand(cmd))
 		return;
 
-	end_input_event();
 	return;
 }
 

src/Instance.cxx

@@ -38,6 +38,7 @@ Instance::Instance()
 		options.password.empty() ? nullptr : options.password.c_str(),
 		*this),
 	 seek(event_loop, client),
+	 update_screen_event(event_loop, BIND_THIS_METHOD(OnUpdateScreen)),
 	 reconnect_timer(event_loop, BIND_THIS_METHOD(OnReconnectTimer)),
 	 update_timer(event_loop, BIND_THIS_METHOD(OnUpdateTimer)),
 #ifndef NCMPC_MINI
@@ -88,6 +89,8 @@ Instance::Run()
 bool
 Instance::OnRawKey(int key) noexcept
 try {
+	update_screen_event.Schedule();
+
 	return screen_manager.OnModalDialogKey(key);
 } catch (...) {
 	screen_manager.status_bar.SetMessage(GetFullMessage(std::current_exception()));

src/Instance.hxx

@@ -10,6 +10,7 @@
 #include "screen.hxx"
 #include "client/mpdclient.hxx"
 #include "event/Loop.hxx"
+#include "event/IdleEvent.hxx"
 #include "event/CoarseTimerEvent.hxx"
 #include "event/FineTimerEvent.hxx"
 
@@ -27,6 +28,8 @@ class Instance final : MpdClientHandler, UserInputHandler {
 
 	DelayedSeek seek;
 
+	IdleEvent update_screen_event;
+
 	/**
 	 * This timer is installed when the connection to the MPD
 	 * server is broken.  It tries to recover by reconnecting
@@ -105,6 +108,8 @@ private:
 	void OnSigwinch() noexcept;
 #endif
 
+	void OnUpdateScreen() noexcept;
+
 	void OnReconnectTimer() noexcept;
 
 	void OnUpdateTimer() noexcept;

src/Main.cxx

@@ -179,19 +179,20 @@ Instance::OnUpdateTimer() noexcept
 		ScheduleUpdateTimer();
 }
 
-void end_input_event() noexcept
+void
+Instance::OnUpdateScreen() noexcept
 {
-	auto &client = global_instance->GetClient();
-
-	screen->Update(client, global_instance->GetSeek());
+	screen_manager.Update(client, seek);
 	client.events = (enum mpd_idle)0;
 
-	auto_update_timer(*global_instance);
+	auto_update_timer(*this);
 }
 
 bool
 Instance::OnCommand(Command cmd) noexcept
 {
+	update_screen_event.Schedule();
+
 	if (cmd == Command::QUIT) {
 		event_loop.Break();
 		return false;
@@ -216,6 +217,8 @@ Instance::OnCommand(Command cmd) noexcept
 bool
 Instance::OnMouse(Point p, mmask_t bstate) noexcept
 {
+	update_screen_event.Schedule();
+
 	try {
 		screen_manager.OnMouse(GetClient(), GetSeek(), p, bstate);
 	} catch (...) {

src/lirc.cxx

@@ -3,7 +3,6 @@
 
 #include "lirc.hxx"
 #include "UserInputHandler.hxx"
-#include "ncmpc.hxx"
 #include "Command.hxx"
 #include "config.h"
 
@@ -21,8 +20,6 @@ LircInput::OnSocketReady(unsigned) noexcept
 				return;
 		}
 	}
-
-	end_input_event();
 }
 
 LircInput::LircInput(EventLoop &_event_loop,

src/ncmpc.hxx

@@ -5,6 +5,3 @@
 
 class ScreenManager;
 extern ScreenManager *screen;
-
-void
-end_input_event() noexcept;