Merge #430

430: Add some initial deny for unused_imports and unused_variables r=bjfish a=bjfish Co-authored-by: Brandon Fish <brandon.j.fish@gmail.com>
wasmerio · May 11, 2019 · 3de17bb · 3de17bb
2 parents fa54025 + 5197d52
commit 3de17bb
Show file tree

Hide file tree

Showing 19 changed files with 28 additions and 7 deletions.
diff --git a/lib/clif-backend/src/lib.rs b/lib/clif-backend/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 mod cache;
 mod func_env;
 mod libcalls;

diff --git a/lib/emscripten/src/lib.rs b/lib/emscripten/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 #[macro_use]
 extern crate wasmer_runtime_core;
 

diff --git a/lib/emscripten/src/syscalls/unix.rs b/lib/emscripten/src/syscalls/unix.rs
@@ -53,7 +53,6 @@ use libc::{
     sendto,
     setpgid,
     setsockopt,
-    size_t,
     sockaddr,
     socket,
     socklen_t,

diff --git a/lib/llvm-backend/src/lib.rs b/lib/llvm-backend/src/lib.rs
@@ -1,3 +1,4 @@
+#![deny(unused_imports, unused_variables)]
 #![cfg_attr(nightly, feature(unwind_attributes))]
 
 mod backend;

diff --git a/lib/llvm-backend/src/platform/unix.rs b/lib/llvm-backend/src/platform/unix.rs
@@ -34,7 +34,7 @@ pub unsafe fn visit_fde(addr: *mut u8, size: usize, visitor: extern "C" fn(*mut
 }
 
 #[cfg(not(target_os = "macos"))]
-pub unsafe fn visit_fde(addr: *mut u8, size: usize, visitor: extern "C" fn(*mut u8)) {
+pub unsafe fn visit_fde(addr: *mut u8, _size: usize, visitor: extern "C" fn(*mut u8)) {
     visitor(addr);
 }
 

diff --git a/lib/middleware-common/src/call_trace.rs b/lib/middleware-common/src/call_trace.rs
@@ -10,7 +10,7 @@ impl FunctionMiddleware for CallTrace {
     fn feed_event<'a, 'b: 'a>(
         &mut self,
         op: Event<'a, 'b>,
-        module_info: &ModuleInfo,
+        _module_info: &ModuleInfo,
         sink: &mut EventSink<'a, 'b>,
     ) -> Result<(), Self::Error> {
         match op {

diff --git a/lib/middleware-common/src/lib.rs b/lib/middleware-common/src/lib.rs
@@ -1 +1,3 @@
+#![deny(unused_imports, unused_variables)]
+
 pub mod call_trace;
diff --git a/lib/runtime-abi/src/lib.rs b/lib/runtime-abi/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 #[cfg(not(target_os = "windows"))]
 #[macro_use]
 extern crate failure;

diff --git a/lib/runtime-c-api/src/lib.rs b/lib/runtime-c-api/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 extern crate wasmer_runtime;
 extern crate wasmer_runtime_core;
 

diff --git a/lib/runtime-core/src/codegen.rs b/lib/runtime-core/src/codegen.rs
@@ -1,6 +1,6 @@
 use crate::{
     backend::RunnableModule,
-    backend::{sys::Memory, Backend, CacheGen, Compiler, CompilerConfig, Token},
+    backend::{Backend, CacheGen, Compiler, CompilerConfig, Token},
     cache::{Artifact, Error as CacheError},
     error::{CompileError, CompileResult},
     module::{ModuleInfo, ModuleInner},

diff --git a/lib/runtime-core/src/lib.rs b/lib/runtime-core/src/lib.rs
@@ -1,3 +1,4 @@
+#![deny(unused_imports, unused_variables)]
 #![cfg_attr(nightly, feature(unwind_attributes))]
 
 #[cfg(test)]

diff --git a/lib/runtime/src/lib.rs b/lib/runtime/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 //! Wasmer-runtime is a library that makes embedding WebAssembly
 //! in your application easy, efficient, and safe.
 //!

diff --git a/lib/singlepass-backend/src/codegen_x64.rs b/lib/singlepass-backend/src/codegen_x64.rs
@@ -478,7 +478,7 @@ impl ModuleCodeGenerator<X64FunctionCode, X64ExecutionContext, CodegenError>
         Ok(())
     }
 
-    unsafe fn from_cache(artifact: Artifact, _: Token) -> Result<ModuleInner, CacheError> {
+    unsafe fn from_cache(_artifact: Artifact, _: Token) -> Result<ModuleInner, CacheError> {
         Err(CacheError::Unknown(
             "the singlepass compiler API doesn't support caching yet".to_string(),
         ))
@@ -1409,7 +1409,7 @@ impl FunctionCodeGenerator<CodegenError> for X64FunctionCode {
         Ok(())
     }
 
-    fn begin_body(&mut self, module_info: &ModuleInfo) -> Result<(), CodegenError> {
+    fn begin_body(&mut self, _module_info: &ModuleInfo) -> Result<(), CodegenError> {
         let a = self.assembler.as_mut().unwrap();
         a.emit_push(Size::S64, Location::GPR(GPR::RBP));
         a.emit_mov(Size::S64, Location::GPR(GPR::RSP), Location::GPR(GPR::RBP));

diff --git a/lib/singlepass-backend/src/lib.rs b/lib/singlepass-backend/src/lib.rs
@@ -1,3 +1,4 @@
+#![deny(unused_imports, unused_variables)]
 #![feature(proc_macro_hygiene)]
 
 #[cfg(not(any(

diff --git a/lib/wasi/src/lib.rs b/lib/wasi/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 #[macro_use]
 extern crate log;
 

diff --git a/lib/win-exception-handler/src/exception_handling.rs b/lib/win-exception-handler/src/exception_handling.rs
@@ -1,4 +1,3 @@
-use std::ffi::c_void;
 use std::ptr::NonNull;
 use wasmer_runtime_core::vm::{Ctx, Func};
 

diff --git a/lib/win-exception-handler/src/lib.rs b/lib/win-exception-handler/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 #[cfg(windows)]
 mod exception_handling;
 

diff --git a/src/bin/wasmer.rs b/src/bin/wasmer.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 extern crate structopt;
 
 use std::env;

diff --git a/src/lib.rs b/src/lib.rs
@@ -1,3 +1,5 @@
+#![deny(unused_imports, unused_variables)]
+
 #[macro_use]
 extern crate wasmer_runtime_core;
 // extern crate wasmer_emscripten;