[studio,turbine] Fix Turbine sleep logic, tweak Studio default sleep values

src/olympic/studio/applib/src/main.cpp

@@ -31,7 +31,7 @@ class StudioUIDrawer: public turbine::gl::Drawer {
 static int updateHandler(turbine::Context &ctx) noexcept {
 	auto sctx = turbine::applicationData<studio::StudioContext>(ctx);
 	sctx->ui.update();
-	return 16;
+	return 1000;
 }
 
 static void keyEventHandler(turbine::Context &ctx, turbine::Key key, bool down) noexcept {
@@ -47,7 +47,7 @@ static ox::Error runApp(
 	turbine::setWindowTitle(*ctx, keelCtx(*ctx).appName);
 	turbine::setUpdateHandler(*ctx, updateHandler);
 	turbine::setKeyEventHandler(*ctx, keyEventHandler);
-	turbine::setConstantRefresh(*ctx, false);
+	turbine::setRefreshWithin(*ctx, 0);
 	StudioUI ui(*ctx, projectDataDir);
 	StudioUIDrawer drawer(ui);
 	turbine::gl::addDrawer(*ctx, &drawer);
@@ -68,7 +68,7 @@ static ox::Error run(
 		static_cast<uint64_t>(time << 1)
 	});
 	// run app
-	auto const err = runApp(appName, projectDataDir, ox::UPtr<ox::FileSystem>(nullptr));
+	auto const err = runApp(appName, projectDataDir, ox::UPtr<ox::FileSystem>{});
 	oxAssert(err, "Something went wrong...");
 	return err;
 }

src/olympic/studio/applib/src/studioapp.cpp

@@ -212,7 +212,6 @@ void StudioUI::drawTabs() noexcept {
 			}
 			if (m_activeEditorOnLastDraw != e.get()) [[unlikely]] {
 				m_activeEditor->onActivated();
-				turbine::setConstantRefresh(m_ctx, m_activeEditor->requiresConstantRefresh());
 			}
 			e->draw(m_sctx);
 			m_activeEditorOnLastDraw = e.get();

src/olympic/turbine/include/turbine/gfx.hpp

@@ -43,6 +43,12 @@ ox::Bounds getWindowBounds(Context &ctx) noexcept;
 
 ox::Error setWindowBounds(Context &ctx, ox::Bounds const&bnds) noexcept;
 
-void setConstantRefresh(Context &ctx, bool r) noexcept;
+/**
+ * Tells Turbine to refresh the screen within the specified period of time.
+ * If the requested value is greater than the current value, the call has no effect.
+ * @param ctx - Context
+ * @param ms - milliseconds
+ */
+void setRefreshWithin(Context &ctx, int ms) noexcept;
 
 }

src/olympic/turbine/src/glfw/context.hpp

@@ -22,8 +22,7 @@ class Context {
 		int uninterruptedRefreshes = 3;
 		ox::UPtr<BaseClipboardObject> clipboard;
 		struct GLFWwindow *window = nullptr;
-		// sets screen refresh to constant instead of only on event
+		int refreshWithinMs = 0;
-		bool constantRefresh = true;
 		ox::Vector<gl::Drawer*, 5> drawers;
 		int64_t startTime = 0;
 		uint64_t wakeupTime = 0;

src/olympic/turbine/src/glfw/gfx.cpp

@@ -260,8 +260,8 @@ ox::Error setWindowBounds(Context &ctx, ox::Bounds const&bnds) noexcept {
 	return {};
 }
 
-void setConstantRefresh(Context &ctx, bool r) noexcept {
+void setRefreshWithin(Context &ctx, int ms) noexcept {
-	ctx.constantRefresh = r;
+	ctx.refreshWithinMs = ox::min(ms, ctx.refreshWithinMs);
 }
 
 }

src/olympic/turbine/src/glfw/turbine.cpp

@@ -69,6 +69,7 @@ static void tickFps(Context &ctx, uint64_t nowMs) noexcept {
 ox::Error run(Context &ctx) noexcept {
 	int sleepTime = 0;
 	while (!glfwWindowShouldClose(ctx.window)) {
+		ctx.refreshWithinMs = 10 * 1000; // refresh within 10 seconds
 		glfwPollEvents();
 		auto const ticks = ticksMs(ctx);
 		if (ctx.wakeupTime <= ticks) {
@@ -79,15 +80,16 @@ ox::Error run(Context &ctx) noexcept {
 				  ctx.wakeupTime = ~uint64_t(0);
 			 }
 		} else {
-			sleepTime = 10;
+			sleepTime = static_cast<int>(ctx.wakeupTime - ticks);
 		}
 		tickFps(ctx, ticks);
 		draw(ctx);
-		if (!ctx.constantRefresh) {
+		auto const realSleepTime = ox::min(ctx.refreshWithinMs, sleepTime);
+		if (realSleepTime) {
 			if (ctx.uninterruptedRefreshes) {
 				--ctx.uninterruptedRefreshes;
 			} else {
-				glfwWaitEventsTimeout(sleepTime);
+				glfwWaitEventsTimeout(static_cast<double>(realSleepTime) / 1000);
 			}
 		}
 	}