Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose panic formatting in API #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kazcw
Copy link

@kazcw kazcw commented May 6, 2022

Hi, I'm working on an Electron application, Enso. Enso needs to do multiple things in our panic hook: We have an application-specific API to submit the backtrace to, but we'd also like to display it with console.error. We want to minimize duplication of the functionality defined in console_error_panic_hook--especially the backtrace formatting, since it addresses browser subtleties.

This proposed change introduces a lower level function to the API, exposing the panic formatting functionality for use from another hook.

(Also, it increments the release version in Cargo.toml to match the currently-released version.)

@wdanilo
Copy link

wdanilo commented May 10, 2022

@alexcrichton @fitzgen do you think it could be merged to this repo? We do not want to keep a local copy of it because it increases the maintenance burden :) I'm pinging you because other PRs are waiting really long time here, so I thought it might be helpful to let you know that this one is opened as well :)

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