Fix deranged-related TryFrom ambiguity

[dtolnay]

When building tarpc in a build graph that also contains deranged, it fails to build. See upstream's related issue jhpratt/deranged#18.

error[E0283]: type annotations needed
   --> tarpc/src/server/limits/channels_per_key.rs:200:26
    |
200 |                 if count >= TryFrom::try_from(*self_.channels_per_key).unwrap() {
    |                          ^^ cannot infer type
    |
    = note: multiple `impl`s satisfying `usize: PartialOrd<_>` found in the following crates: `core`, `deranged`:
            - impl PartialOrd for usize;
            - impl<MIN, MAX> PartialOrd<RangedUsize<MIN, MAX>> for usize
              where the constant `MIN` has type `usize`, the constant `MAX` has type `usize`;

[tikue]

Appreciate it!

[dtolnay]

Rustfmt job failure and clippy job failure are both unrelated to this PR.

Diff in /home/runner/work/tarpc/tarpc/tarpc/src/server/request_hook/before.rs:157:
 impl<Req, First: BeforeRequest<Req>, Rest: BeforeRequestList<Req>> BeforeRequestList<Req>
     for BeforeRequestCons<First, Rest>
 {
-    type Then<Next> = BeforeRequestCons<First, Rest::Then<Next>> where Next: BeforeRequest<Req>;
+    type Then<Next>
+        = BeforeRequestCons<First, Rest::Then<Next>>
+    where
+        Next: BeforeRequest<Req>;
 
     fn then<Next: BeforeRequest<Req>>(self, next: Next) -> Self::Then<Next> {
         let BeforeRequestCons(first, rest) = self;
Diff in /home/runner/work/tarpc/tarpc/tarpc/src/server/request_hook/before.rs:172:
 }
 
 impl<Req> BeforeRequestList<Req> for BeforeRequestNil {
-    type Then<Next> = BeforeRequestCons<Next, BeforeRequestNil> where Next: BeforeRequest<Req>;
+    type Then<Next>
+        = BeforeRequestCons<Next, BeforeRequestNil>
+    where
+        Next: BeforeRequest<Req>;
 
     fn then<Next: BeforeRequest<Req>>(self, next: Next) -> Self::Then<Next> {
         BeforeRequestCons(next, BeforeRequestNil)

error: the following explicit lifetimes could be elided: 'a
   --> plugins/src/lib.rs:535:6
    |
535 | impl<'a> ServiceGenerator<'a> {
    |      ^^                   ^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
    = note: `-D clippy::needless-lifetimes` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::needless_lifetimes)]`
help: elide the lifetimes
    |
535 - impl<'a> ServiceGenerator<'a> {
535 + impl ServiceGenerator<'_> {
    |

    Checking http v1.3.1
error: the following explicit lifetimes could be elided: 'a
   --> plugins/src/lib.rs:811:6
    |
811 | impl<'a> ToTokens for ServiceGenerator<'a> {
    |      ^^                                ^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
help: elide the lifetimes
    |
811 - impl<'a> ToTokens for ServiceGenerator<'a> {
811 + impl ToTokens for ServiceGenerator<'_> {
    |

[dtolnay]

Rebased over #516 and #517.

[dtolnay]

Would it be possible to get this in a crates.io release relatively soon? I expect a nontrivial fraction of tarpc users depend on the troublesome deranged dependency and are currently broken, because it's a dependency of the widely used time crate and pulled in by rustls, tracing, zip, etc.

[tikue]

Would it be possible to get this in a crates.io release relatively soon? I expect a nontrivial fraction of tarpc users depend on the troublesome deranged dependency and are currently broken, because it's a dependency of the widely used time crate and pulled in by rustls, tracing, zip, etc.

Done, let me know if you have any issues. Thanks!