Removed uneeded TODOs.

Author: illegalprime

src/client/builder.rs

@@ -55,9 +55,6 @@ mod async_imports {
 #[cfg(feature="async")]
 use self::async_imports::*;
 
-// TODO: add extra funcs for future stuff, like auto ping and auto close
-
-
 /// Build clients with a builder-style API
 /// This makes it easy to create and configure a websocket
 /// connection:
@@ -662,7 +659,6 @@ impl<'u> ClientBuilder<'u> {
 		Box::new(future)
 	}
 
-	// TODO: add timeout option for connecting
 	// TODO: add conveniences like .response_to_pings, .send_close, etc.
 	/// Asynchronously create an insecure (plain TCP) connection to the client.
 	///
@@ -974,6 +970,4 @@ mod tests {
 		assert!(protos.contains(&"electric".to_string()));
 		assert!(!protos.contains(&"rust-websocket".to_string()));
 	}
-
-	// TODO: a few more
 }

src/codec/ws.rs

@@ -24,9 +24,6 @@ use ws::message::Message as MessageTrait;
 use ws::util::header::read_header;
 use result::WebSocketError;
 
-// TODO: IMPORTANT: check if frame_size is correct,
-// do not call .reserve with the entire size
-
 /// Even though a websocket connection may look perfectly symmetrical
 /// in reality there are small differences between clients and servers.
 /// This type is passed to the codecs to inform them of what role they are in
@@ -290,8 +287,6 @@ impl<M> Encoder for MessageCodec<M>
 	}
 }
 
-// TODO: add tests to check boundary cases for reading dataframes
-
 #[cfg(test)]
 mod tests {
 	use super::*;

src/dataframe.rs

@@ -61,7 +61,6 @@ impl DataFrame {
 				if !should_be_masked {
 					return Err(WebSocketError::DataFrameError("Expected unmasked data frame"));
 				}
-				// TODO: move data to avoid copy?
 				mask::mask_data(mask, &body)
 			}
 			None => {

src/receiver.rs

@@ -132,6 +132,6 @@ impl ws::Receiver for Receiver {
 			}
 		}
 
-    Ok(::std::mem::replace(&mut self.buffer, Vec::new()))
+		Ok(::std::mem::replace(&mut self.buffer, Vec::new()))
 	}
 }