adopt for_str as only constructor (and rename it for_url)

Author: samstokes

src/client.rs

@@ -126,23 +126,8 @@ impl Client {
     ///
     /// [`ClientBuilder`]: struct.ClientBuilder.html
     /// [`.stream()`]: #method.stream
-    pub fn for_url<U: r::IntoUrl>(url: U) -> Result<ClientBuilder> {
-        let url = url
-            .into_url()
-            .map_err(|e| Error::HttpRequest(Box::new(e)))?;
-        Ok(ClientBuilder {
-            url: url,
-            headers: r::header::HeaderMap::new(),
-        })
-    }
-
-    /// Construct a new `Client` (via a [`ClientBuilder`]). This will not
-    /// perform any network activity until [`.stream()`] is called.
-    ///
-    /// [`ClientBuilder`]: struct.ClientBuilder.html
-    /// [`.stream()`]: #method.stream
-    pub fn for_str(str_url: &str) -> Result<ClientBuilder> {
-        let url = Url::parse(str_url).map_err(|e| Error::HttpRequest(Box::new(e)))?;
+    pub fn for_url(url: &str) -> Result<ClientBuilder> {
+        let url = Url::parse(url).map_err(|e| Error::HttpRequest(Box::new(e)))?;
         Ok(ClientBuilder {
             url: url,
             headers: r::header::HeaderMap::new(),