Skip to content

Commit

Permalink
Remove trailing whitespace with URL in make_url at once
Browse files Browse the repository at this point in the history
  • Loading branch information
sammykim committed Jan 3, 2014
1 parent be8ed32 commit 586de9b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/components/util/url.rs
Expand Up @@ -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(_) => {
Expand Down

5 comments on commit 586de9b

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from SimonSapin
at sammykim@586de9b

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sammykim/servo/load_url = 586de9b into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 528d362

Please sign in to comment.