diff options
| author | Vito Caputo <vcaputo@pengaru.com> | 2023-05-11 12:48:42 -0700 |
|---|---|---|
| committer | Vito Caputo <vcaputo@pengaru.com> | 2023-05-11 15:19:25 -0700 |
| commit | 2b9eca8724fe6c5e0f6a4438cd398b5978283131 (patch) | |
| tree | 6e78ff7b7269926ddda6eb944cc2c97237e11b97 /src/modules/swarm | |
| parent | caa75150ab4982c5ca0e48e606dd4f81087e7a3d (diff) | |
modules/*: remove use of static default setups
There was a time when it made sense for context creates needing
setups but not receiving them to still be functional with some
sane defaults.
But with recursive settings, we really shouldn't ever have
orphaned nested module uses unreachable by a proper setup.
So let's just get rid of this fallback, and exclusively rely on
the baked setups provided by the .setup() methods. They still
have preferred defaults, and the proper setup production
machinery is what should be responsible for applying those
at runtime where they may also be overridden or otherwise
influenced.
Diffstat (limited to 'src/modules/swarm')
| -rw-r--r-- | src/modules/swarm/swarm.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/modules/swarm/swarm.c b/src/modules/swarm/swarm.c index adb8df8..8883197 100644 --- a/src/modules/swarm/swarm.c +++ b/src/modules/swarm/swarm.c @@ -65,10 +65,6 @@ typedef struct swarm_context_t { #define SWARM_ZCONST 4.f #define SWARM_DEFAULT_STYLE SWARM_DRAW_STYLE_LINES -static swarm_setup_t swarm_default_setup = { - .draw_style = SWARM_DEFAULT_STYLE, -}; - static inline float randf(unsigned *seed, float min, float max) { @@ -185,9 +181,6 @@ static til_module_context_t * swarm_create_context(const til_module_t *module, t { swarm_context_t *ctxt; - if (!setup) - setup = &swarm_default_setup.til_setup; - ctxt = til_module_context_new(module, sizeof(swarm_context_t) + sizeof(*(ctxt->boids)) * SWARM_SIZE, stream, seed, ticks, n_cpus, path); if (!ctxt) return NULL; |