Tidy the preFrame logic into a single function in SceneRenderer

Clients now just call that one helper and SceneProvider passes in all
the renderables.

1 files changed, 10 insertions, 5 deletions

diff --git a/test/test-assetFactory.cpp b/test/test-assetFactory.cpp
index ff38369..8dbe53d 100644
--- a/test/test-assetFactory.cpp
+++ b/test/test-assetFactory.cpp
@@ -69,16 +69,21 @@ namespace {
 		}
 
 		void
-		render(float dist)
+		forEachRenderable(const RenderableProcessor & func) const override
 		{
-			sceneRenderer.camera.setView({-dist, dist * 1.2F, dist * 1.2F}, south + east + down);
-			const auto & [camFrust, lightFrust] = sceneRenderer.preFrame({{0.9, 0.5}});
 			for (const auto & [assetId, asset] : gameState.assets) {
 				if (const auto renderable = asset.getAs<Renderable>()) {
-					renderable->preFrame(camFrust, lightFrust);
+					func(renderable);
 				}
 			}
-			gameState.world.apply<Renderable>(&Renderable::preFrame, camFrust, lightFrust);
+			gameState.world.apply<Renderable>(func);
+		}
+
+		void
+		render(float dist)
+		{
+			sceneRenderer.camera.setView({-dist, dist * 1.2F, dist * 1.2F}, south + east + down);
+			sceneRenderer.preFrame(*this, {{0.9, 0.5}});
 			sceneRenderer.render(*this);
 		}