search

http-rs / http-types

Common types for HTTP operations
https://docs.rs/http-types
Apache License 2.0
200 stars 84 forks source link

Remove Clone from StatusCode #323

Closed Fishrock123 closed 3 years ago

Fishrock123 commented 3 years ago

Mentioned in https://github.com/http-rs/http-types/pull/319#issuecomment-761530241 StatusCode is Clone, but it's also Copy, as it is just a wrapper around u16. I think we should remove Clone from it since it doesn't really make sense.

yoshuawuyts commented 3 years ago

Copy has a requirement on Clone. It's defined as:

pub trait Copy: Clone { }

We can't remove Clone for that reason.