diff options
| author | Felipe Contreras <felipe.contreras@gmail.com> | 2021-05-15 16:21:03 -0500 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2021-05-17 07:25:14 -0300 |
| commit | 5f49e3421f5d1f98951b55dd5ba106c2a106aba7 (patch) | |
| tree | a52f14c69d24bb2b2b53151e4eb46975c4783e19 /bindings/ruby/init.c | |
| parent | 9574fb60991fa81799fbd1d26d41693ca2794e6b (diff) | |
ruby: new notmuch_rb_object_destroy() helper
The struct used to store the types (rb_data_type_t) contains a "data"
field where we can store whatever we want. I use that field to store a
pointer to the corresponding destroy function. For example
notmuch_rb_database_type contains a pointer to notmuch_database_destroy.
I cast that pointer as a notmuch_status_t (func*)(void *) and call
that function passing the internal object (e.g. notmuch_database_t).
Using the rb_data_type_t data we can call the correct notmuch destroy
function.
Therefore this:
ret = ((notmuch_status_t (*)(void *)) type->data) (nm_object);
Is effectively the same as this:
ret = notmuch_database_destroy (database);
The advantage of doing it this way is that much less code is necesary
since each rb_data_type_t has the corresponding destroy function stored
in it.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Diffstat (limited to 'bindings/ruby/init.c')
| -rw-r--r-- | bindings/ruby/init.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/bindings/ruby/init.c b/bindings/ruby/init.c index a9f863eb..62515eca 100644 --- a/bindings/ruby/init.c +++ b/bindings/ruby/init.c @@ -54,6 +54,7 @@ const rb_data_type_t notmuch_rb_object_type = { const rb_data_type_t notmuch_rb_ ## id ## _type = { \ .wrap_struct_name = "notmuch_" #id, \ .parent = ¬much_rb_object_type, \ + .data = ¬much_ ## id ## _destroy, \ } define_type (database); |
