summaryrefslogtreecommitdiff
path: root/src/effects/particles_effect.cc
diff options
context:
space:
mode:
authorskal <pascal.massimino@gmail.com>2026-02-17 13:05:11 +0100
committerskal <pascal.massimino@gmail.com>2026-02-17 13:05:11 +0100
commitbdb1a4e95a545f3f4d88630b8aec6ab771776d99 (patch)
tree97a3a15381a7ff9780644ec8876535dd219f23cf /src/effects/particles_effect.cc
parenta32c7456588abf4f44866d0c055fa94d105e8ef7 (diff)
refactor(effects): Streamline uniforms initialization
Centralized uniforms_buffer_ initialization and updates to Effect base class: - init_uniforms_buffer() now automatic in Effect::Effect() - uniforms_buffer_.update() now automatic in dispatch_render() - Removed redundant calls from all effect subclasses - Updated effect.h comments to reflect automatic behavior - Updated EFFECT_WORKFLOW.md templates Benefits: - 16 lines removed from effect implementations - Consistent pattern enforced at compile time - Reduced boilerplate for new effects Tests: 34/34 passing handoff(Claude): Effect base class now handles uniforms automatically
Diffstat (limited to 'src/effects/particles_effect.cc')
-rw-r--r--src/effects/particles_effect.cc5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/effects/particles_effect.cc b/src/effects/particles_effect.cc
index 3c9feb7..d0336f6 100644
--- a/src/effects/particles_effect.cc
+++ b/src/effects/particles_effect.cc
@@ -14,8 +14,6 @@ Particles::Particles(const GpuContext& ctx,
: Effect(ctx, inputs, outputs, start_time, end_time) {
HEADLESS_RETURN_IF_NULL(ctx_.device);
- init_uniforms_buffer();
-
// Initialize particles buffer
std::vector<Particle> init_p(NUM_PARTICLES);
for (int i = 0; i < NUM_PARTICLES; ++i) {
@@ -66,9 +64,6 @@ Particles::Particles(const GpuContext& ctx,
void Particles::render(WGPUCommandEncoder encoder,
const UniformsSequenceParams& params,
NodeRegistry& nodes) {
- // Update uniforms
- uniforms_buffer_.update(ctx_.queue, params);
-
// Run compute pass (particle simulation)
WGPUComputePassEncoder compute =
wgpuCommandEncoderBeginComputePass(encoder, nullptr);