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.

Sign up for GitHub

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

Jump to bottom

Make panic handler optional #177

Closed
3than3 opened this issue Oct 17, 2020 · 2 comments
Closed

Make panic handler optional #177

3than3 opened this issue Oct 17, 2020 · 2 comments
Labels
enhancement good first issue help wanted

Comments

@3than3
Copy link
Contributor

3than3 commented Oct 17, 2020 •
edited
Loading

Currently, the uefi-services crate contains a panic handler, while this is useful, It is a problem if you are doing printing yourself as you can't use your own printer for panics. The best option would be to make the panic handler optional using cfg_attr()

Thank you for your time!
@notrodes
Copy link

notrodes commented Feb 6, 2021

I think this can be closed because of #179

@GabrielMajeri
Copy link
Collaborator

Yep, closing now. Thanks for the heads-up.

@ivila ivila mentioned this issue Oct 24, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@GabrielMajeri @notrodes @3than3