From 6ebb5ba17e9b84312ccaaba031d33a2406b6059d Mon Sep 17 00:00:00 2001 From: Vito Caputo Date: Tue, 13 Jun 2023 22:39:29 -0700 Subject: til: til_module_setup() needs to finalize when no .setup() Commit 24911280 made til_setup_t required, and most setup code has already adopted til_module_setup_finalize() added in c94e4683 for this purpose. But til_module_setup()'s fall-through case when there's no til_module_t.setup() on the module being setup was missed. This broke running simple modules without a .setup() directly like --module=roto, oops. Fix is simple. --- src/til.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/til.c') diff --git a/src/til.c b/src/til.c index fd9b167..fec6b46 100644 --- a/src/til.c +++ b/src/til.c @@ -511,7 +511,7 @@ int til_module_setup(const til_settings_t *settings, til_setting_t **res_setting if (module->setup) return module->setup(settings, res_setting, res_desc, res_setup); - return 0; + return til_module_setup_finalize(module, settings, res_setup); } -- cgit v1.2.1