til: pass module to .context_create()/til_module_context_new()

Let's make it so til_module_context_t as returned from
til_module_context_new() can immediately be freed via
til_module_context_free().

Previously it was only after the context propagated out to
til_module_context_create() that it could be freed that way, as
that was where the module member was being assigned.

With this change, and wiring up the module pointer into
til_module_t.create_context() as well for convenient providing to
til_module_context_new(), til_module_t.create_context() error
paths can easily cleanup via `return til_module_context_free()`

But this does require the til_module_t.destroy_context() be able
to safely handle partially constructed contexts, since the
mid-create failure freeing won't necessarily have all the members
initialized.  There will probably be some NULL derefs to fix up,
but at least the contexts are zero-initialized @ new.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/til_module_context.h b/src/til_module_context.h
index 6f4339b..a11b876 100644
--- a/src/til_module_context.h
+++ b/src/til_module_context.h
@@ -17,7 +17,7 @@ struct til_module_context_t {
 	uint32_t		path_hash;
 };
 
-void * til_module_context_new(til_stream_t *stream, size_t size, unsigned seed, unsigned ticks, unsigned n_cpus, char *path);
+void * til_module_context_new(const til_module_t *module, size_t size, til_stream_t *stream, unsigned seed, unsigned ticks, unsigned n_cpus, char *path);
 void * til_module_context_free(til_module_context_t *module_context);
 
 #endif