Skip to content
Permalink
Browse files

Remove trailing whitespace with URL in make_url at once

  • Loading branch information...
sammykim committed Jan 3, 2014
1 parent be8ed32 commit 586de9ba8eea6d841ca3df3c8f2b4c764ebb0f9f
Showing with 1 addition and 0 deletions.
  1. +1 −0 src/components/util/url.rs
@@ -18,6 +18,7 @@ Create a URL object from a string. Does various helpful browsery things like
*/
// TODO: about:failure->
pub fn make_url(str_url: ~str, current_url: Option<Url>) -> Url {
let str_url = str_url.trim_chars(& &[' ', '\t', '\n', '\r', '\x0C']).to_owned();
let schm = url::get_scheme(str_url);
let str_url = match schm {
Err(_) => {

5 comments on commit 586de9b

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jan 3, 2014

saw approval from SimonSapin
at sammykim@586de9b

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jan 3, 2014

merging sammykim/servo/load_url = 586de9b into auto

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jan 3, 2014

sammykim/servo/load_url = 586de9b merged ok, testing candidate = 528d362

@bors-servo

This comment has been minimized.

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jan 3, 2014

fast-forwarding master to auto = 528d362

Please sign in to comment.
You can’t perform that action at this time.