Add dyn keyword to all trait objects

Author: jtgeibel

src/lib.rs

@@ -104,7 +104,7 @@ impl conduit::Request for ConduitRequest {
         conduit::Scheme::Http
     }
 
-    fn headers(&self) -> &conduit::Headers {
+    fn headers(&self) -> &dyn conduit::Headers {
         &self.parts
     }
 
@@ -152,7 +152,7 @@ impl conduit::Request for ConduitRequest {
         self.parts.0.uri.query()
     }
 
-    fn body(&mut self) -> &mut Read {
+    fn body(&mut self) -> &mut dyn Read {
         &mut self.body
     }
 }
@@ -214,7 +214,7 @@ impl<H: conduit::Handler> hyper::service::NewService for Service<H> {
     type ResBody = Body;
     type Error = hyper::Error;
     type Service = Service<H>;
-    type Future = Box<Future<Item = Self::Service, Error = Self::InitError> + Send>;
+    type Future = Box<dyn Future<Item = Self::Service, Error = Self::InitError> + Send>;
     type InitError = hyper::Error;
 
     fn new_service(&self) -> Self::Future {
@@ -226,7 +226,7 @@ impl<H: conduit::Handler> hyper::service::Service for Service<H> {
     type ReqBody = Body;
     type ResBody = Body;
     type Error = hyper::Error;
-    type Future = Box<Future<Item = Response<Self::ResBody>, Error = Self::Error> + Send>;
+    type Future = Box<dyn Future<Item = Response<Self::ResBody>, Error = Self::Error> + Send>;
 
     /// Returns a future which buffers the response body and then calls the conduit handler from a thread pool
     fn call(&mut self, request: Request<Self::ReqBody>) -> Self::Future {

src/tests.rs

@@ -8,7 +8,7 @@ use hyper;
 
 struct OkResult;
 impl Handler for OkResult {
-    fn call(&self, _req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, _req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         Ok(Response {
             status: (200, "OK"),
             headers: build_headers("value"),
@@ -19,22 +19,22 @@ impl Handler for OkResult {
 
 struct ErrorResult;
 impl Handler for ErrorResult {
-    fn call(&self, _req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, _req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         let error = ::std::io::Error::last_os_error();
         Err(Box::new(error))
     }
 }
 
 struct Panic;
 impl Handler for Panic {
-    fn call(&self, _req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, _req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         panic!()
     }
 }
 
 struct InvalidHeader;
 impl Handler for InvalidHeader {
-    fn call(&self, _req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, _req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         let mut headers = build_headers("discarded");
         headers.insert("invalid".into(), vec!["\r\n".into()]);
         Ok(Response {
@@ -47,7 +47,7 @@ impl Handler for InvalidHeader {
 
 struct InvalidStatus;
 impl Handler for InvalidStatus {
-    fn call(&self, _req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, _req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         Ok(Response {
             status: (1000, "invalid status code"),
             headers: build_headers("discarded"),
@@ -58,7 +58,7 @@ impl Handler for InvalidStatus {
 
 struct AssertPathNormalized;
 impl Handler for AssertPathNormalized {
-    fn call(&self, req: &mut Request) -> Result<Response, Box<Error + Send>> {
+    fn call(&self, req: &mut dyn Request) -> Result<Response, Box<dyn Error + Send>> {
         if req.path() == "/normalized" {
             OkResult.call(req)
         } else {