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

Consider having a more friendly way to display errors and warnings #2296

Closed
pvdrz opened this issue Oct 5, 2022 · 2 comments
Closed

Consider having a more friendly way to display errors and warnings #2296

pvdrz opened this issue Oct 5, 2022 · 2 comments

Comments

@pvdrz
Copy link
Contributor

pvdrz commented Oct 5, 2022

This is more of a proposal than a proper issue report.

I've considered using a crate like miette or annotate-snippets to build something similar to how rustc stores and emit diagnostic messages.

Some of the warning log entries that bindgen currently emits could be displayed as diagnostics for both the library and CLI and I think this would seriously improve the user experience. In particular when those diagnostics could include the actual c code that triggered it when possible.

Of course this would require some effort and I'd volunteer to do it myself if you consider this is worth it.

cc: @emilio @kulp @fitzgen @amanjeev

@amanjeev
Copy link
Member

I am always up for improving ergonomics of error and warning reporting. I wonder if we can do some work to identify which major areas we need this? Then, it makes easier for us to see get a project roadmap, create issues for the work to be done etc.

@pvdrz
Copy link
Contributor Author

pvdrz commented Feb 16, 2023

Linking this to keep track of issues mentioning this topic: #1768 (comment)

@pvdrz pvdrz closed this as completed Apr 14, 2023
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

No branches or pull requests

2 participants