Skip to content

Commit

Permalink
Fix compiletest trim deprecation warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
phansch committed Dec 19, 2018
1 parent 748d354 commit 036ce5c
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
10 changes: 5 additions & 5 deletions src/tools/compiletest/src/header.rs
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ impl EarlyProps {
fn ignore_lldb(config: &Config, line: &str) -> bool {
if let Some(ref actual_version) = config.lldb_version {
if line.starts_with("min-lldb-version") {
let min_version = line.trim_right()
let min_version = line.trim_end()
.rsplit(' ')
.next()
.expect("Malformed lldb version directive");
Expand All @@ -228,15 +228,15 @@ impl EarlyProps {
}
if let Some(ref actual_version) = config.llvm_version {
if line.starts_with("min-llvm-version") {
let min_version = line.trim_right()
let min_version = line.trim_end()
.rsplit(' ')
.next()
.expect("Malformed llvm version directive");
// Ignore if actual version is smaller the minimum required
// version
&actual_version[..] < min_version
} else if line.starts_with("min-system-llvm-version") {
let min_version = line.trim_right()
let min_version = line.trim_end()
.rsplit(' ')
.next()
.expect("Malformed llvm version directive");
Expand Down Expand Up @@ -573,14 +573,14 @@ fn iter_header(testfile: &Path, cfg: Option<&str>, it: &mut dyn FnMut(&str)) {
None => false,
};
if matches {
it(ln[(close_brace + 1)..].trim_left());
it(ln[(close_brace + 1)..].trim_start());
}
} else {
panic!("malformed condition directive: expected `{}foo]`, found `{}`",
comment_with_brace, ln)
}
} else if ln.starts_with(comment) {
it(ln[comment.len() ..].trim_left());
it(ln[comment.len() ..].trim_start());
}
}
return;
Expand Down
10 changes: 5 additions & 5 deletions src/tools/compiletest/src/runtest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1079,7 +1079,7 @@ impl<'test> TestCx<'test> {
match line {
Ok(line) => {
let line = if line.starts_with("//") {
line[2..].trim_left()
line[2..].trim_start()
} else {
line.as_str()
};
Expand Down Expand Up @@ -2143,8 +2143,8 @@ impl<'test> TestCx<'test> {
.lines()
.enumerate()
.filter_map(|(line_nb, line)| {
if (line.trim_left().starts_with("pub mod ")
|| line.trim_left().starts_with("mod "))
if (line.trim_start().starts_with("pub mod ")
|| line.trim_start().starts_with("mod "))
&& line.ends_with(';')
{
if let Some(ref mut other_files) = other_files {
Expand All @@ -2153,7 +2153,7 @@ impl<'test> TestCx<'test> {
None
} else {
let sline = line.split("///").last().unwrap_or("");
let line = sline.trim_left();
let line = sline.trim_start();
if line.starts_with("```") {
if ignore {
ignore = false;
Expand Down Expand Up @@ -3284,7 +3284,7 @@ fn normalize_mir_line(line: &str) -> String {
fn nocomment_mir_line(line: &str) -> &str {
if let Some(idx) = line.find("//") {
let (l, _) = line.split_at(idx);
l.trim_right()
l.trim_end()
} else {
line
}
Expand Down

0 comments on commit 036ce5c

Please sign in to comment.