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

digest: use impl Trait notation for AsRef<[u8]> #112

Merged
merged 1 commit into from
May 23, 2020

Conversation

tarcieri
Copy link
Member

Replaces generic parameters on methods which take an AsRef<[u8]> bound with impl AsRef<[u8]>.

This syntax is semantically equivalent but eliminates a generic parameter, which (IMO) makes the method signature easier to read, and in general seems to be preferred after the introduction of impl Trait.

Replaces generic parameters on methods which take an `AsRef<[u8]>` bound
with `impl AsRef<[u8]>`.

This syntax is semantically equivalent but eliminates a generic
parameter, which (IMO) makes the method signature easier to read, and in
general seems to be preferred after the introduction of `impl Trait`.
@tarcieri tarcieri requested a review from newpavlov May 23, 2020 15:48
@tarcieri tarcieri merged commit c7aa80d into master May 23, 2020
@tarcieri tarcieri deleted the digest/use-impl-trait branch May 23, 2020 15:52
@tarcieri tarcieri mentioned this pull request Jun 10, 2020
dns2utf8 pushed a commit to dns2utf8/traits that referenced this pull request Jan 24, 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

Successfully merging this pull request may close these issues.

1 participant