Remove all warnings when running doc tests

Warning that had previously been buried by not supplying `--no-capture`
to `cargo test` have been addressed here.

Author: bradleybeddoes

src/handler/mod.rs

@@ -38,13 +38,13 @@ impl<T> NewHandlerService<T>
     /// # extern crate hyper;
     /// # extern crate borrow_bag;
     /// #
-    /// # use gotham::handler::{NewHandlerService, NewHandler, Handler};
+    /// # use gotham::handler::NewHandlerService;
     /// # use gotham::state::State;
     /// # use hyper::server::{Request, Response};
     /// # use hyper::StatusCode;
     /// #
     /// # fn main() {
-    /// fn handler(state: State, request: Request) -> (State, Response) {
+    /// fn handler(state: State, _req: Request) -> (State, Response) {
     ///     (state, Response::new().with_status(StatusCode::Accepted))
     /// }
     ///
@@ -59,7 +59,7 @@ impl<T> NewHandlerService<T>
     /// # extern crate hyper;
     /// # extern crate borrow_bag;
     /// #
-    /// # use gotham::handler::{NewHandlerService, NewHandler, Handler};
+    /// # use gotham::handler::NewHandlerService;
     /// # use gotham::state::State;
     /// # use gotham::router::Router;
     /// # use gotham::router::tree::Tree;
@@ -70,7 +70,7 @@ impl<T> NewHandlerService<T>
     /// # use hyper::{StatusCode, Method};
     /// #
     /// # fn main() {
-    /// fn handler(state: State, request: Request) -> (State, Response) {
+    /// fn handler(state: State, _req: Request) -> (State, Response) {
     ///     (state, Response::new().with_status(StatusCode::Accepted))
     /// }
     ///
@@ -225,11 +225,10 @@ impl IntoHandlerFuture for Box<HandlerFuture> {
 /// # use gotham::router::tree::Tree;
 /// # use gotham::router::request_matcher::MethodOnlyRequestMatcher;
 /// # use gotham::dispatch::Dispatcher;
-/// # use gotham::handler::{NewHandler, IntoResponse};
-/// # use futures::{future, Future};
+/// # use gotham::handler::IntoResponse;
 /// # use hyper::Method;
 /// # use hyper::StatusCode;
-/// # use hyper::server::{Http, Request, Response};
+/// # use hyper::server::{Request, Response};
 /// #
 /// struct MyStruct {
 ///     value: String
@@ -250,7 +249,7 @@ impl IntoHandlerFuture for Box<HandlerFuture> {
 ///     }
 /// }
 ///
-/// fn handler(state: State, req: Request) -> (State, MyStruct) {
+/// fn handler(state: State, _req: Request) -> (State, MyStruct) {
 ///     (state, MyStruct::new())
 /// }
 ///

src/middleware/mod.rs

@@ -21,10 +21,10 @@ pub mod pipeline;
 /// # extern crate gotham;
 /// # extern crate hyper;
 /// #
-/// # use gotham::handler::{Handler, HandlerFuture};
+/// # use gotham::handler::HandlerFuture;
 /// # use gotham::middleware::Middleware;
 /// # use gotham::state::State;
-/// # use hyper::server::{Request, Response};
+/// # use hyper::server::Request;
 /// #
 /// struct NoopMiddleware;
 ///
@@ -36,7 +36,9 @@ pub mod pipeline;
 ///     }
 /// }
 /// #
-/// # fn main() {}
+/// # fn main() {
+/// #  NoopMiddleware {};
+/// # }
 /// ```
 ///
 /// Recording a piece of state data before passing the request through:
@@ -45,13 +47,14 @@ pub mod pipeline;
 /// # extern crate gotham;
 /// # extern crate hyper;
 /// #
-/// # use gotham::handler::{Handler, HandlerFuture};
+/// # use gotham::handler::HandlerFuture;
 /// # use gotham::middleware::Middleware;
 /// # use gotham::state::{State, StateData};
-/// # use hyper::server::{Request, Response};
+/// # use hyper::server::Request;
 /// #
 /// struct MiddlewareWithStateData;
 ///
+/// # #[allow(unused)]
 /// struct MiddlewareStateData {
 ///     i: i32,
 /// }
@@ -67,7 +70,9 @@ pub mod pipeline;
 ///     }
 /// }
 /// #
-/// # fn main() {}
+/// # fn main() {
+/// #     MiddlewareWithStateData {};
+/// # }
 /// ```
 ///
 /// Terminating the request early based on some arbitrary condition:
@@ -77,9 +82,9 @@ pub mod pipeline;
 /// # extern crate hyper;
 /// # extern crate futures;
 /// #
-/// # use gotham::handler::{Handler, HandlerFuture};
+/// # use gotham::handler::HandlerFuture;
 /// # use gotham::middleware::Middleware;
-/// # use gotham::state::{State, StateData};
+/// # use gotham::state::{State};
 /// # use hyper::server::{Request, Response};
 /// # use hyper::{Method, StatusCode};
 /// # use futures::{future, Future};
@@ -99,7 +104,9 @@ pub mod pipeline;
 ///     }
 /// }
 /// #
-/// # fn main() {}
+/// # fn main() {
+/// #     ConditionalMiddleware {};
+/// # }
 /// ```
 ///
 /// Asynchronous middleware, which continues the request after some action completes:
@@ -109,10 +116,10 @@ pub mod pipeline;
 /// # extern crate hyper;
 /// # extern crate futures;
 /// #
-/// # use gotham::handler::{Handler, HandlerFuture};
+/// # use gotham::handler::HandlerFuture;
 /// # use gotham::middleware::Middleware;
 /// # use gotham::state::State;
-/// # use hyper::server::{Request, Response};
+/// # use hyper::server::Request;
 /// # use futures::{future, Future};
 /// #
 /// struct AsyncMiddleware;
@@ -128,7 +135,9 @@ pub mod pipeline;
 ///     }
 /// }
 /// #
-/// # fn main() {}
+/// # fn main() {
+///     AsyncMiddleware {};
+/// # }
 /// ```
 pub trait Middleware {
     /// Entry point to the middleware. To pass the request on to the application, the middleware

src/middleware/pipeline.rs

@@ -211,11 +211,10 @@ pub fn new_pipeline() -> PipelineBuilder<()> {
 /// #
 /// # use std::io;
 /// # use gotham::state::State;
-/// # use gotham::handler::{Handler, HandlerFuture};
+/// # use gotham::handler::HandlerFuture;
 /// # use gotham::middleware::{Middleware, NewMiddleware};
-/// # use gotham::middleware::pipeline::{new_pipeline, Pipeline, PipelineBuilder};
-/// # use hyper::server::{Request, Response};
-/// # use hyper::StatusCode;
+/// # use gotham::middleware::pipeline::new_pipeline;
+/// # use hyper::server::Request;
 /// #
 /// # #[derive(Clone)]
 /// # struct MiddlewareOne;
@@ -269,12 +268,8 @@ pub fn new_pipeline() -> PipelineBuilder<()> {
 /// #   }
 /// # }
 /// #
-/// # fn handler(state: State, _: Request) -> (State, Response) {
-/// #   (state, Response::new().with_status(StatusCode::Accepted))
-/// # }
-/// #
 /// # fn main() {
-/// let pipeline: Pipeline<_> = new_pipeline()
+/// new_pipeline()
 ///     .add(MiddlewareOne)
 ///     .add(MiddlewareTwo)
 ///     .add(MiddlewareThree)

src/router/tree/node.rs

@@ -429,7 +429,6 @@ mod tests {
                        .segment(),
                    "seg7");
 
-        println!("~~~");
         // GET /some/path/seg9/another/path
         assert_eq!(root.traverse(&["/", "some", "path", "seg9", "some2", "path2"])
                        .unwrap()