- Notifications
You must be signed in to change notification settings - Fork10
Fixes for debug formatting#22
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
dcsommer commentedAug 14, 2025
- Debug now prints full values for the internal representation, which is more helpful when combined with a hex or binary debug printing mode.
- Display now properly renders all the scripts with " + " separators.
Manishearth commentedAug 14, 2025
I'm opposed to retargeting the Debug impl. Most of the time someone using this crate cares will need one so they can slap derive(Debug) on a struct containing it, and as such clients of this crate should never have to care about the internal implementation. A debug impl that exposes the internal bitfields is primarily useful for people working on this crate itself, which is not a great tradeoff: if I'm working on this crete I can always temporarily tweak its Debug impl if I need. |
`Debug` and `Display` now properly print all the scripts with " + " seperators.
dcsommer commentedAug 14, 2025
Fair enough. Limited the PR to bug fix for |
a5f747b intounicode-rs:masterUh oh!
There was an error while loading.Please reload this page.