Merge pull request #19 from paritytech/non-determinism-check

take module as ref in is_deterministic

nondeterminism_check/src/main.rs

@@ -17,7 +17,7 @@ fn main() {
 	// Loading module
 	let module = parity_wasm::deserialize_file(&args[1]).expect("Module deserialization to succeed");
 
-	if wasm_utils::is_deterministic(module) {
+	if wasm_utils::is_deterministic(&module) {
 		println!("Module is deterministic");
 	} else {
 		println!("Module is not deterministic");

src/nondeterminism_check.rs

@@ -77,7 +77,7 @@ fn have_nondeterministic_opcodes (opcodes: &[Opcode]) -> bool {
 
 
 
-pub fn is_deterministic(module: elements::Module) -> bool {
+pub fn is_deterministic(module: &elements::Module) -> bool {
 	for section in module.sections() {
 		match *section {
 			Section::Code(ref cs) => {
@@ -114,7 +114,7 @@ mod tests {
 				.build()
 			.build()
 		.build();
-		assert_eq!(false, is_deterministic(module));
+		assert_eq!(false, is_deterministic(&module));
 	}
 
 	#[test]
@@ -133,6 +133,6 @@ mod tests {
 				.build()
 			.build()
 		.build();
-		assert_eq!(true, is_deterministic(module));
+		assert_eq!(true, is_deterministic(&module));
 	}
 }