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

iced::font::Error should implement std::error::Error #2049

Open
2 tasks done
Cookie04DE opened this issue Aug 24, 2023 · 3 comments
Open
2 tasks done

iced::font::Error should implement std::error::Error #2049

Cookie04DE opened this issue Aug 24, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@Cookie04DE
Copy link

Is there an existing issue for this?

  • I have searched the existing issues.

Is this issue related to iced?

  • My hardware is compatible and my graphics drivers are up-to-date.

What happened?

I tried using iced::font::Error as a dyn std::error::Error and noticed that the trait is not implemented.

What is the expected behavior?

iced::font::Error should implement std::error::Error since it is an error.

Version

crates.io release

Operating System

Windows

Do you have any log output?

No response

@Cookie04DE Cookie04DE added the bug Something isn't working label Aug 24, 2023
@thunderstorm010
Copy link
Contributor

Well, its also an empty Enum which means that it can't be constructed (at least safely). So no errors!?

@Cookie04DE
Copy link
Author

Huh, weird, but you're right. Guess it's currently impossible for font loading to fail.
Nevertheless, if this is gonna change, the enum should start implementing the Error trait.

@thunderstorm010
Copy link
Contributor

thunderstorm010 commented Aug 25, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants