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

Remove stabilized features #25

Merged
merged 1 commit into from
Sep 30, 2018
Merged

Conversation

josephlr
Copy link
Contributor

panic_implemenation was renamed to panic_handler:
   rust-lang/rust#44489 (comment)
panic_handler was stablized:
   rust-lang/rust#51366

`cargo check` now succeeds without warnings
@josephlr josephlr closed this Sep 11, 2018
@josephlr josephlr reopened this Sep 11, 2018
@michaelmelanson
Copy link

Any update on this pull request? I'm getting an error about the panic_implementation rename to panic_handler that I think this would solve.

@phil-opp phil-opp merged commit c2ca02d into rust-osdev:master Sep 30, 2018
@phil-opp
Copy link
Member

I'm sorry for the delay! I've been on vacation for the last few weeks. I will publish a new version as soon as I get back to my computer. Thanks for the PR @josephlr!

@michaelmelanson
Copy link

michaelmelanson commented Sep 30, 2018 via email

@phil-opp
Copy link
Member

phil-opp commented Oct 1, 2018

I released version 0.3.2 with this change.

@josephlr josephlr deleted the warnings branch October 8, 2018 20:33
Nickforall pushed a commit to Nickforall/bootloader that referenced this pull request Dec 17, 2018
panic_implemenation was renamed to panic_handler:
   rust-lang/rust#44489 (comment)
panic_handler was stablized:
   rust-lang/rust#51366

`cargo check` now succeeds without warnings
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.

3 participants