Skip to content

android: make set_panic_hook configurable using a Platform conf setting.#565

Merged
not-fl3 merged 1 commit into
not-fl3:masterfrom
narodnik:master
Jun 30, 2025
Merged

android: make set_panic_hook configurable using a Platform conf setting.#565
not-fl3 merged 1 commit into
not-fl3:masterfrom
narodnik:master

Conversation

@narodnik

Copy link
Copy Markdown
Contributor

Apologies my last commit commented out the panic hook override by accident.

I normally disable this locally because I want the backtrace and my own format.

So I made it configurable.

BTW do we want the backtrace in the default miniquad output? I find it more useful than a simple single line crash. Here's what I use:

fn panic_hook(panic_info: &std::panic::PanicHookInfo) {
    error!("panic occurred: {panic_info}");
    error!("{}", std::backtrace::Backtrace::force_capture().to_string());
    std::process::abort()
}

Anyways this is my PR

@not-fl3 not-fl3 merged commit 5575f03 into not-fl3:master Jun 30, 2025
11 checks passed
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.

2 participants