output/Interface: the default Pause() implementation fails

Fixes regression from commit 31bad5f7: if Pause() returns true, the output thread is running a busy loop, causing 100% CPU usage, instead of just closing the output. Closes #105

output/Interface: the default Pause() implementation fails
91438b65 · Max Kellermann · 987c5059 · 91438b65
Commit 91438b65 authored Aug 31, 2017 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

Interface.hxx src/output/Interface.hxx +4 -1

No files found.
--- a/src/output/Interface.hxx
+++ b/src/output/Interface.hxx
@@ -141,7 +141,10 @@ public:
 	 * @return false on error (output will be closed by caller),
 	 * true for continue to pause
 	 */
-	virtual bool Pause() noexcept { return true; }
+	virtual bool Pause() noexcept {
+		/* fail because this method is not implemented */
+		return false;
+	}
 };

 #endif