jngl::updateInput() calls Audio::step() so that play/stop don't deadl…

…ock when using audio::engine::Impl::DummyImpl

src/audio/sdl/engine.cpp

@@ -4,7 +4,6 @@
 // https://lisyarus.github.io/blog/programming/2022/10/15/audio-mixing.html
 #include "../engine.hpp"
 
-#include "../../jngl/debug.hpp"
 #include "../../jngl/other.hpp"
 #include "../../log.hpp"
 #include "../Stream.hpp"
@@ -22,7 +21,7 @@ struct engine::Impl {
 		try {
 			backend = std::make_unique<SdlImpl>(output);
 		} catch (std::exception& e) {
-			debugLn(e.what());
+			internal::warn(e.what());
 			backend = std::make_unique<DummyImpl>(std::move(output));
 		}
 	}

src/window.cpp

@@ -251,6 +251,13 @@ void Window::updateKeyStates() {
 		}
 		updateInputCallbacks.clear();
 	}
+
+	// this is done here, because Window::updateKeyStates() is called by jngl::updateInput() and if
+	// someone writes their own main loop, Audio::step() needs to be called. Maybe updateKeyStates()
+	// should be renamed to updateInput() and UpdateInput() moved to WindowImpl ...
+	if (auto audio = Audio::handleIfAlive()) {
+		audio->step();
+	}
 }
 
 double Window::getMouseWheel() const {
@@ -364,9 +371,6 @@ void Window::stepIfNeeded() {
 #ifdef JNGL_PERFORMANCE_OVERLAY
 		auto start = std::chrono::steady_clock::now();
 #endif
-		if (auto audio = Audio::handleIfAlive()) {
-			audio->step();
-		}
 		for (auto& job : jobs) {
 			job->step();
 		}