Revert "event/Thread: start the thread in the constructor"

This reverts commit b49cfe96f406bf0b750b7a1ea73609905d606335.  It was
a bad idea because it broke signal handlers.  I need to find a better
way to fix io_uring intialization.

src/Main.cxx

@@ -329,6 +329,9 @@ MainConfigured(const CommandLineOptions &options,
 	Instance instance;
 	global_instance = &instance;
 
+	instance.io_thread.Start();
+	instance.rtio_thread.Start();
+
 #ifdef ENABLE_NEIGHBOR_PLUGINS
 	instance.neighbors = std::make_unique<NeighborGlue>();
 	instance.neighbors->Init(raw_config,

src/event/Thread.hxx

@@ -20,9 +20,7 @@ class EventThread final {
 public:
 	explicit EventThread(bool _realtime=false)
 		:event_loop(ThreadId::Null()), thread(BIND_THIS_METHOD(Run)),
-		 realtime(_realtime) {
-		Start();
-	}
+		 realtime(_realtime) {}
 
 	~EventThread() noexcept {
 		Stop();
@@ -32,11 +30,11 @@ public:
 		return event_loop;
 	}
 
-private:
 	void Start();
 
 	void Stop() noexcept;
 
+private:
 	void Run() noexcept;
 };
 

test/DumpDatabase.cxx

@@ -28,6 +28,12 @@ class GlobalInit {
 	EventThread io_thread;
 
 public:
+	GlobalInit() {
+		io_thread.Start();
+	}
+
+	~GlobalInit() = default;
+
 	EventLoop &GetEventLoop() {
 		return io_thread.GetEventLoop();
 	}

test/DumpOgg.cxx

@@ -24,6 +24,7 @@ try {
 	const char *path = argv[1];
 
 	EventThread io_thread;
+	io_thread.Start();
 
 	const ScopeInputPluginsInit input_plugins_init(ConfigData(),
 						       io_thread.GetEventLoop());

test/RunChromaprint.cxx

@@ -84,6 +84,8 @@ public:
 		 input_plugins_init(config, io_thread.GetEventLoop()),
 		 decoder_plugins_init(config)
 	{
+		io_thread.Start();
+
 		pcm_convert_global_init(config);
 	}
 };

test/dump_playlist.cxx

@@ -47,6 +47,7 @@ try {
 	const auto config = AutoLoadConfigFile(config_path);
 
 	EventThread io_thread;
+	io_thread.Start();
 
 	const ScopeInputPluginsInit input_plugins_init(config, io_thread.GetEventLoop());
 	const ScopePlaylistPluginsInit playlist_plugins_init(config);

test/dump_text_file.cxx

@@ -34,6 +34,7 @@ public:
 	explicit GlobalInit(Path config_path)
 		:config(AutoLoadConfigFile(config_path))
 	{
+		io_thread.Start();
 	}
 };
 

test/read_tags.cxx

@@ -86,6 +86,7 @@ try {
 	const char *path = argv[2];
 
 	EventThread io_thread;
+	io_thread.Start();
 
 	const ScopeInputPluginsInit input_plugins_init(ConfigData(),
 						       io_thread.GetEventLoop());

test/run_decoder.cxx

@@ -91,6 +91,7 @@ public:
 		 input_plugins_init(config, io_thread.GetEventLoop()),
 		 decoder_plugins_init(config)
 	{
+		io_thread.Start();
 	}
 };
 

test/run_input.cxx

@@ -130,6 +130,7 @@ public:
 	explicit GlobalInit(Path config_path)
 		:config(AutoLoadConfigFile(config_path))
 	{
+		io_thread.Start();
 	}
 };
 

test/run_output.cxx

@@ -158,6 +158,7 @@ try {
 	const auto config = AutoLoadConfigFile(c.config_path);
 
 	EventThread io_thread;
+	io_thread.Start();
 
 	/* initialize the audio output */
 

test/run_storage.cxx

@@ -114,6 +114,7 @@ public:
 	GlobalInit(Path config_path)
 		:config(AutoLoadConfigFile(config_path))
 	{
+		io_thread.Start();
 	}
 
 	EventLoop &GetEventLoop() noexcept {

test/visit_archive.cxx

@@ -35,6 +35,7 @@ public:
 	explicit GlobalInit(Path config_path)
 		:config(AutoLoadConfigFile(config_path))
 	{
+		io_thread.Start();
 	}
 };