From bd939acdf750181ef0e1a612b445da4c15077c85 Mon Sep 17 00:00:00 2001 From: skal Date: Sat, 7 Feb 2026 17:04:56 +0100 Subject: refactor: Bundle GPU context into GpuContext struct - Created GpuContext struct {device, queue, format} - Updated Effect/PostProcessEffect to take const GpuContext& - Updated all 19 effect implementations - Updated MainSequence.init() and LoadTimeline() signatures - Updated generated timeline files - Updated all test files - Added gpu_get_context() accessor and fixture.ctx() helper Fixes test_mesh.cc compilation error from g_device/g_queue/g_format conflicts. All targets build successfully. --- src/tests/test_effect_base.cc | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/tests/test_effect_base.cc') diff --git a/src/tests/test_effect_base.cc b/src/tests/test_effect_base.cc index bc837be..8180535 100644 --- a/src/tests/test_effect_base.cc +++ b/src/tests/test_effect_base.cc @@ -73,7 +73,7 @@ static void test_effect_construction() { // Create FlashEffect (simple post-process effect) auto effect = std::make_shared( - fixture.device(), fixture.queue(), fixture.format()); + fixture.ctx()); assert(!effect->is_initialized && "Effect should not be initialized yet"); @@ -92,11 +92,11 @@ static void test_effect_initialization() { // Create MainSequence (use init_test for test environment) MainSequence main_seq; - main_seq.init_test(fixture.device(), fixture.queue(), fixture.format()); + main_seq.init_test(fixture.ctx()); // Create FlashEffect auto effect = std::make_shared( - fixture.device(), fixture.queue(), fixture.format()); + fixture.ctx()); assert(!effect->is_initialized && "Effect should not be initialized yet"); @@ -123,14 +123,14 @@ static void test_sequence_add_effect() { } MainSequence main_seq; - main_seq.init_test(fixture.device(), fixture.queue(), fixture.format()); + main_seq.init_test(fixture.ctx()); // Create sequence auto seq = std::make_shared(); // Create effect auto effect = std::make_shared( - fixture.device(), fixture.queue(), fixture.format()); + fixture.ctx()); assert(!effect->is_initialized && "Effect should not be initialized before Sequence::init()"); @@ -156,11 +156,11 @@ static void test_sequence_activation() { } MainSequence main_seq; - main_seq.init_test(fixture.device(), fixture.queue(), fixture.format()); + main_seq.init_test(fixture.ctx()); auto seq = std::make_shared(); auto effect = std::make_shared( - fixture.device(), fixture.queue(), fixture.format()); + fixture.ctx()); // Effect active from 5.0 to 10.0 seconds seq->add_effect(effect, 5.0f, 10.0f, 0); -- cgit v1.2.3