diff options
author | Vito Caputo <vcaputo@pengaru.com> | 2023-08-04 19:51:13 -0700 |
---|---|---|
committer | Vito Caputo <vcaputo@pengaru.com> | 2023-08-04 19:51:13 -0700 |
commit | 72e9eb1188e4889f23c50836d1ccd2206852e588 (patch) | |
tree | af9b54c6d2358b3ea5cef972a99b72af79d3b64f /src/modules/strobe | |
parent | 802e366db7049f1d8fdab6f38f968bcf0c55b3ae (diff) |
modules/rkt: utilize til_module_setup_full()
rkt_setup() and rkt_scener_update() had distinct implementations
for scene module setup. This consolidates that where trivial to
both use the new til_module_setup_full() with appropriate
parameters, wrapped up in rkt_scene_module_setup().
The finalizing phase is still ad-hoc which is mildly annoying,
but if finalizing just passed into rkt_scene_module_setup() there
wouldn't be the til_module_t onhand for sticking in rkt_scene_t.
So the code to extract and lookup the module from the settings
would still be needed anyways, as the whole til setup_func api
isn't limited to modules so the baked til_setup_t doesn't come
back with a til_module_t hanging in there. Maybe in the future
this gets changed a bit, there could for instance be a void* in
til_setup_t where something usage-specific goes, like the
relevant module in the case of a module's setup. Something to
consider for the future.
Consolidating these in the pre-finalize phase at least ensures
consistent behavior in initial rkt::scenes setup vs. scener
editing/new scenes.
Diffstat (limited to 'src/modules/strobe')
0 files changed, 0 insertions, 0 deletions