From adaeb10b5bec3e5a344fbcd90c27204ecb221c80 Mon Sep 17 00:00:00 2001 From: ljedrz Date: Thu, 6 Dec 2018 16:39:19 +0100 Subject: codegen_utils, driver: fix clippy errors --- src/librustc_codegen_utils/codegen_backend.rs | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) (limited to 'src/librustc_codegen_utils') diff --git a/src/librustc_codegen_utils/codegen_backend.rs b/src/librustc_codegen_utils/codegen_backend.rs index 046f0cccfee..b328873800e 100644 --- a/src/librustc_codegen_utils/codegen_backend.rs +++ b/src/librustc_codegen_utils/codegen_backend.rs @@ -86,8 +86,8 @@ impl MetadataLoader for NoLlvmMetadataLoader { let mut buf = Vec::new(); io::copy(&mut file, &mut buf).unwrap(); - let buf: OwningRef, [u8]> = OwningRef::new(buf).into(); - return Ok(rustc_erase_owner!(buf.map_owner_box())); + let buf: OwningRef, [u8]> = OwningRef::new(buf); + Ok(rustc_erase_owner!(buf.map_owner_box())) } fn get_dylib_metadata(&self, target: &Target, filename: &Path) -> Result { @@ -103,7 +103,7 @@ pub struct OngoingCodegen { } impl MetadataOnlyCodegenBackend { - pub fn new() -> Box { + pub fn boxed() -> Box { box MetadataOnlyCodegenBackend(()) } } @@ -165,15 +165,12 @@ impl CodegenBackend for MetadataOnlyCodegenBackend { tcx, collector::MonoItemCollectionMode::Eager ).0 { - match mono_item { - MonoItem::Fn(inst) => { - let def_id = inst.def_id(); - if def_id.is_local() { - let _ = inst.def.is_inline(tcx); - let _ = tcx.codegen_fn_attrs(def_id); - } + if let MonoItem::Fn(inst) = mono_item { + let def_id = inst.def_id(); + if def_id.is_local() { + let _ = inst.def.is_inline(tcx); + let _ = tcx.codegen_fn_attrs(def_id); } - _ => {} } } tcx.sess.abort_if_errors(); @@ -181,7 +178,7 @@ impl CodegenBackend for MetadataOnlyCodegenBackend { let metadata = tcx.encode_metadata(); box OngoingCodegen { - metadata: metadata, + metadata, metadata_version: tcx.metadata_encoding_version().to_vec(), crate_name: tcx.crate_name(LOCAL_CRATE), } -- cgit 1.4.1-3-g733a5