better handler result handling

examples/basic.rs

@@ -78,10 +78,10 @@ fn main() {
             .resource("/", |r| r.handler(Method::GET, |req| {
                 println!("{:?}", req);
 
-                Ok(httpcodes::HTTPFound
+                httpcodes::HTTPFound
                     .build()
                     .header("LOCATION", "/index.html")
-                   .body(Body::Empty)?)
+                    .body(Body::Empty)
             }))
             // static files
             .route_handler("/static", StaticFiles::new("examples/static/", true)))

src/application.rs

@@ -148,9 +148,7 @@ impl<S> ApplicationBuilder<S> where S: 'static {
     ///     let app = Application::default("/")
     ///         .resource("/test", |r| {
     ///              r.get::<MyRoute>();
-    ///              r.handler(Method::HEAD, |req| {
+    ///              r.handler(Method::HEAD, |req| httpcodes::HTTPMethodNotAllowed);
-    ///                  Ok(httpcodes::HTTPMethodNotAllowed)
-    ///              });
     ///         })
     ///         .finish();
     /// }

src/httpresponse.rs

@@ -205,11 +205,11 @@ impl HttpResponse {
 }
 
 /// Helper conversion implementation
-impl<I: Into<HttpResponse>, E: Into<HttpResponse>> From<Result<I, E>> for HttpResponse {
+impl<I: Into<HttpResponse>, E: Into<Error>> From<Result<I, E>> for HttpResponse {
     fn from(res: Result<I, E>) -> Self {
         match res {
             Ok(val) => val.into(),
-            Err(err) => err.into(),
+            Err(err) => err.into().into(),
         }
     }
 }

tests/test_server.rs

@@ -16,9 +16,7 @@ fn create_server<T, A>() -> HttpServer<T, A, Application<()>> {
     HttpServer::new(
         vec![Application::default("/")
              .resource("/", |r|
-                       r.handler(Method::GET, |_| {
+                       r.handler(Method::GET, |_| httpcodes::HTTPOk))
-                           Ok(httpcodes::HTTPOk)
-                       }))
              .finish()])
 }
 
@@ -96,9 +94,7 @@ fn test_middlewares() {
                                             response: act_num2,
                                             finish: act_num3})
                  .resource("/", |r|
-                           r.handler(Method::GET, |_| {
+                           r.handler(Method::GET, |_| httpcodes::HTTPOk))
-                               Ok(httpcodes::HTTPOk)
-                           }))
                  .finish()])
             .serve::<_, ()>("127.0.0.1:58904").unwrap();
         sys.run();