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
Cleaner header construction with the field_init_shorthand feature #92
Comments
Ho ! Sorry, you want only stable Rust. We will probably wait this feature to be added into stable releases. |
Yes, and that's some of my oldest Rust code. Could definitely use some touch ups... |
Rust 1.17 was released and it supports the field init shortcut syntax now. BTW, I saw the comment in the code about using the builder pattern. I agree that the current, non-builder, way is better. |
Yeah, I just saw that today. That will be really nice. If anyone wants to submit a PR for this, that would be great. Otherwise, I'll start cleaning some of this up as I come across it. |
fixed in #179 |
https://github.com/bluejekyll/trust-dns/blob/2b705dc3001c369cd75446996cc10c8f673a35c1/client/src/op/header.rs#L289-L298
The
field_init_shorthand
unstable feature can probably help you get a cleaner header construction.The text was updated successfully, but these errors were encountered: