Handle functions with multiple callbacks #367

Merged

jwharm merged 2 commits from notified-scope into main

2026-05-25 09:58:40 +02:00

jwharm commented

2026-05-23 20:28:08 +02:00

Owner

This also fixes the detection of DestroyNotify parameters, which means some function signatures have improved.

Also added a metadata fix for g_file_move_async to properly annotate the progress_callback scope.

Fixes #366

Some functions have multiple callbacks with notified or async scope, but one of them has no DestroyNotify argument of its own. Java-GI used to allocate these in a global arena, leaking the allocated memory. To fix this, we now use the same arena for both callback allocations. This also fixes the detection of DestroyNotify parameters, which means some function signatures have improved. Also added a metadata fix for `g_file_move_async` to properly annotate the `progress_callback` scope. Fixes #366

jwharm added 1 commit

2026-05-23 20:28:09 +02:00

Handle functions with multiple callbacks d307e2b2f5

Some functions have multiple callbacks with notified or async scope,
but one of them has no DestroyNotify argument of its own. Java-GI used
to allocate these in a global arena, leaking the allocated memory. To
fix this, we now use the same arena for both callback allocations.

This also fixes the detection of DestroyNotify parameters, which means
some function signatures have improved.

Also added a metadata fix for `g_file_move_async` to properly annotate
the `progress_callback` scope.

Fixes #366

jwharm referenced this pull request

2026-05-23 20:33:14 +02:00

SIGSEGV in upcall_stub_load_target when async callbacks are generated with Arena.ofConfined() #366

jwharm added 1 commit

2026-05-25 09:57:48 +02:00

Add tests for GFile move_async and copy_async 657bc2c7f8

jwharm merged commit 582e1b860f into main

2026-05-25 09:58:40 +02:00

jwharm referenced this pull request from a commit

2026-05-25 09:58:42 +02:00

Merge pull request 'Handle functions with multiple callbacks' (#367) from notified-scope into main