Remove module_from_bytes helper

chisel/src/main.rs

@@ -309,7 +309,7 @@ fn execute_module(context: &ModuleContext, module: &mut Module) -> bool {
 
 fn chisel_execute(context: &ChiselContext) -> Result<bool, &'static str> {
     if let Ok(buffer) = read(context.file()) {
-        if let Ok(module) = module_from_bytes(&buffer) {
+        if let Ok(module) = Module::from_bytes(&buffer) {
             // If we do not parse the NamesSection here, parity-wasm will drop it at serialisation
             // It is useful to have this for a number of optimisation passes, including binaryenopt and snip
             // TODO: better error handling

libchisel/src/lib.rs

@@ -28,11 +28,6 @@ pub mod verifyimports;
 
 mod depgraph;
 
-// This helper is exported here for users of this library not needing to import parity_wasm.
-pub fn module_from_bytes<T: AsRef<[u8]>>(input: T) -> Result<Module, ModuleError> {
-    Ok(Module::from_bytes(input)?)
-}
-
 #[derive(Eq, PartialEq, Debug)]
 pub enum ModuleKind {
     Creator,
@@ -250,17 +245,4 @@ mod tests {
         let result = as_trait.validate(&Module::default());
         assert!(result.is_ok());
     }
-
-    #[test]
-    fn loading_from_bytes() {
-        let module_orig = Module::default();
-
-        let output = module_orig.clone().to_bytes().unwrap();
-        let module = module_from_bytes(&output).unwrap();
-        assert_eq!(module_orig, module);
-
-        let output = module_orig.clone().to_bytes().unwrap();
-        let module = Module::from_bytes(&output).unwrap();
-        assert_eq!(module_orig, module);
-    }
 }