Skip to content

Use todo instead of unimplemented#1723

Merged
senekor merged 1 commit intomainfrom
unimpl-todo
Sep 14, 2023
Merged

Use todo instead of unimplemented#1723
senekor merged 1 commit intomainfrom
unimpl-todo

Conversation

@senekor
Copy link
Contributor

@senekor senekor commented Sep 10, 2023

The difference is one of intent, and the actual panic message.
See the docs why todo is more appropriate:
https://doc.rust-lang.org/stable/std/macro.todo.html

[no important files changed]

closes #1598

The difference is one of intent, and the actual panic message.
See the docs why todo is more appropriate:
https://doc.rust-lang.org/stable/std/macro.todo.html

[no important files changed]

closes #1598
Copy link
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great

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.

(Nitpick) Use todo! instead of unimplemented!.

2 participants