search

briansmith / ring

Safe, fast, small crypto using Rust
Other
3.76k stars 708 forks source link

Replace `std::error` with `core::error` #2118

Open StackOverflowExcept1on opened 5 months ago

StackOverflowExcept1on commented 5 months ago

https://github.com/briansmith/ring/blob/525047a2b4f3724fd1337b3fc8a7c642c95fcf8b/src/error.rs#L86-L87

https://github.com/rust-lang/rust/pull/125951

briansmith commented 5 months ago

We can add a core-error feature flag to ring that will do this, I suppose. Not sure what the use case is.

StackOverflowExcept1on commented 5 months ago

Well, maybe use case is to not depend on std and be able to use crates like thiserror

briansmith commented 4 months ago

This idea seems fine with me, but it'd need to be behind a feature flag because we aren't raising the MSRV yet.