Skip to content

Conversation

@npmccallum
Copy link
Contributor

No description provided.

@npmccallum npmccallum force-pushed the misc branch 3 times, most recently from c484094 to c8bca8d Compare January 9, 2023 17:45
@tarcieri
Copy link
Member

tarcieri commented Jan 9, 2023

Re: TryFrom<&[u8]>, I think it's nice to have on types which represent the toplevel SEQUENCEs of a document. I agree it doesn't have to be everywhere.

The advantage over Decode::from_der is in the 2021 edition TryFrom is in the prelude, so it skips an import.

@baloo it'd be good if you can take a look at this to see if it overlaps with what you've been working on

@baloo
Copy link
Member

baloo commented Jan 9, 2023

no overlap that I can see, all of it makes sense.

These just duplicate the Decode trait and are inconsistently implemented.

Signed-off-by: Nathaniel McCallum <[email protected]>
Signed-off-by: Nathaniel McCallum <[email protected]>
RustCrypto#806

The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.

Signed-off-by: Nathaniel McCallum <[email protected]>
This crate unconditionally requires alloc.

Signed-off-by: Nathaniel McCallum <[email protected]>
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.

3 participants