summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorEmilio Cobos Álvarez <emilio@crisal.io>2018-03-04 14:43:13 +0100
committerEmilio Cobos Álvarez <emilio@crisal.io>2018-03-04 14:43:13 +0100
commitcfd153cc64c3212e3a19e78837f3a45884975641 (patch)
tree2e70d0804b597ec4581ede9a16c28b6506993a28 /tests
parentd5ca4e7e90a3a5ebfeff1041c7eaeb7596d9c1a5 (diff)
Untry.
Use the ? operator instead of try, and add some more uses of it on Option<> that were straight-forward.
Diffstat (limited to 'tests')
-rw-r--r--tests/tests.rs18
1 files changed, 9 insertions, 9 deletions
diff --git a/tests/tests.rs b/tests/tests.rs
index 5b12861f..6ee67fed 100644
--- a/tests/tests.rs
+++ b/tests/tests.rs
@@ -97,10 +97,10 @@ fn compare_generated_header(
header: &PathBuf,
builder: Builder,
) -> Result<(), Error> {
- let file_name = try!(header.file_name().ok_or(Error::new(
+ let file_name = header.file_name().ok_or(Error::new(
ErrorKind::Other,
"compare_generated_header expects a file",
- )));
+ ))?;
let mut expectation = PathBuf::from(header);
expectation.pop();
@@ -162,7 +162,7 @@ fn compare_generated_header(
let mut expected = String::new();
{
if let Ok(expectation_file) = fs::File::open(&expectation) {
- try!(BufReader::new(expectation_file).read_to_string(&mut expected));
+ BufReader::new(expectation_file).read_to_string(&mut expected)?;
}
}
@@ -195,22 +195,22 @@ fn compare_generated_header(
// Override the diff.
{
- let mut expectation_file = try!(fs::File::create(&expectation));
- try!(expectation_file.write_all(actual.as_bytes()));
+ let mut expectation_file = fs::File::create(&expectation)?;
+ expectation_file.write_all(actual.as_bytes())?;
}
Err(Error::new(ErrorKind::Other, "Header and binding differ!"))
}
fn create_bindgen_builder(header: &PathBuf) -> Result<Option<Builder>, Error> {
- let source = try!(fs::File::open(header));
+ let source = fs::File::open(header)?;
let reader = BufReader::new(source);
// Scoop up bindgen-flags from test header
let mut flags = Vec::with_capacity(2);
for line in reader.lines() {
- let line = try!(line);
+ let line = line?;
if !line.starts_with("// bindgen") {
continue;
}
@@ -249,10 +249,10 @@ fn create_bindgen_builder(header: &PathBuf) -> Result<Option<Builder>, Error> {
// - add header filename as 1st element
// - prepend raw lines so they're in the right order for expected output
// - append the test header's bindgen flags
- let header_str = try!(header.to_str().ok_or(Error::new(
+ let header_str = header.to_str().ok_or(Error::new(
ErrorKind::Other,
"Invalid header file name",
- )));
+ ))?;
let prepend = [
"bindgen",