Skip to content

Commit

Permalink
Fully initialize custom_operations job_ops
Browse files Browse the repository at this point in the history
It was missing an initializer for `.compare_ext` (since 3.12.1), the
default is `custom_compare_ext_default`; and `.fixed_length` (since
4.08.0) for which `NULL` is good.
  • Loading branch information
MisterDA committed Jun 21, 2022
1 parent 449f180 commit 8dacb11
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
10 changes: 8 additions & 2 deletions src/unix/lwt_libev_stubs.c
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,10 @@ static long hash_loop(value loop) { return (long)Ev_loop_val(loop); }

static struct custom_operations loop_ops = {
"lwt.libev.loop", custom_finalize_default, compare_loops,
hash_loop, custom_serialize_default, custom_deserialize_default};
hash_loop, custom_serialize_default, custom_deserialize_default,
custom_compare_ext_default,
NULL
};

/* Do nothing.
Expand Down Expand Up @@ -125,7 +128,10 @@ static long hash_watcher(value watcher) { return (long)Ev_io_val(watcher); }

static struct custom_operations watcher_ops = {
"lwt.libev.watcher", custom_finalize_default, compare_watchers,
hash_watcher, custom_serialize_default, custom_deserialize_default};
hash_watcher, custom_serialize_default, custom_deserialize_default,
custom_compare_ext_default,
NULL
};

/* +-----------------------------------------------------------------+
| IO watchers |
Expand Down
5 changes: 4 additions & 1 deletion src/unix/lwt_unix_stubs.c
Original file line number Diff line number Diff line change
Expand Up @@ -1021,7 +1021,10 @@ static void *worker_loop(void *data) {
/* Description of jobs. */
struct custom_operations job_ops = {
"lwt.unix.job", custom_finalize_default, custom_compare_default,
custom_hash_default, custom_serialize_default, custom_deserialize_default};
custom_hash_default, custom_serialize_default, custom_deserialize_default,
custom_compare_ext_default,
NULL
};

/* Get the job structure contained in a custom value. */
#define Job_val(v) *(lwt_unix_job *)Data_custom_val(v)
Expand Down

0 comments on commit 8dacb11

Please sign in to comment.