Skip to content

add release(Bool) option to process_wait/3#3310

Open
Skgland wants to merge 4 commits into
mthom:masterfrom
Skgland:process_wait-release
Open

add release(Bool) option to process_wait/3#3310
Skgland wants to merge 4 commits into
mthom:masterfrom
Skgland:process_wait-release

Conversation

@Skgland
Copy link
Copy Markdown
Contributor

@Skgland Skgland commented Apr 25, 2026

This changes the behavior of process_wait/2 and process_wait/3.

The default for release(Bool) is release(true) matching sictus, but the old behavior was that of release(false).

The added tests run into #3300 due to the asserta at Line 184 of the toplevel

( catch(user:Goal,
Exception,
( write_term(Goal, [variable_names(VNs),double_quotes(DQ)]),
write(' causes: '),
write_term(Exception, [double_quotes(DQ)]), nl,
asserta(g_caused_exception(Goal, Exception))
)
) -> true
; write('% Warning: initialization failed for: '),
write_term(Goal, [variable_names(VNs),double_quotes(DQ)]), nl
),

cc #3308

Skgland added 4 commits April 25, 2026 14:35
make process_wait/3 release the Process by default
old rust versions didn't print the pid
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant