Minor clippy fix.

windelbouwman · windelbouwman · commit 7bb6f8fdaa3f · 2019-03-02T11:22:40.000+01:00
diff --git a/vm/src/bytecode.rs b/vm/src/bytecode.rs
@@ -190,7 +190,7 @@ pub enum Constant {
     Boolean { value: bool },
     String { value: String },
     Bytes { value: Vec<u8> },
-    Code { code: CodeObject },
+    Code { code: Box<CodeObject> },
     Tuple { elements: Vec<Constant> },
     None,
 }
diff --git a/vm/src/compile.rs b/vm/src/compile.rs
@@ -603,7 +603,9 @@ impl Compiler {
 
         self.prepare_decorators(decorator_list)?;
         self.emit(Instruction::LoadConst {
-            value: bytecode::Constant::Code { code },
+            value: bytecode::Constant::Code {
+                code: Box::new(code),
+            },
         });
         self.emit(Instruction::LoadConst {
             value: bytecode::Constant::String {
@@ -653,7 +655,9 @@ impl Compiler {
 
         let code = self.pop_code_object();
         self.emit(Instruction::LoadConst {
-            value: bytecode::Constant::Code { code },
+            value: bytecode::Constant::Code {
+                code: Box::new(code),
+            },
         });
         self.emit(Instruction::LoadConst {
             value: bytecode::Constant::String {
@@ -1075,7 +1079,9 @@ impl Compiler {
                 self.emit(Instruction::ReturnValue);
                 let code = self.pop_code_object();
                 self.emit(Instruction::LoadConst {
-                    value: bytecode::Constant::Code { code },
+                    value: bytecode::Constant::Code {
+                        code: Box::new(code),
+                    },
                 });
                 self.emit(Instruction::LoadConst {
                     value: bytecode::Constant::String { value: name },
@@ -1360,7 +1366,9 @@ impl Compiler {
 
         // List comprehension code:
         self.emit(Instruction::LoadConst {
-            value: bytecode::Constant::Code { code },
+            value: bytecode::Constant::Code {
+                code: Box::new(code),
+            },
         });
 
         // List comprehension function name:
diff --git a/vm/src/frame.rs b/vm/src/frame.rs
@@ -716,8 +716,7 @@ impl Frame {
         let obj = import_module(vm, current_path, module)?;
 
         for (k, v) in obj.get_key_value_pairs().iter() {
-            &self
-                .scope
+            self.scope
                 .locals
                 .set_item(&vm.ctx, &objstr::get_value(k), v.clone());
         }
diff --git a/vm/src/pyobject.rs b/vm/src/pyobject.rs
@@ -682,7 +682,7 @@ impl PyContext {
             bytecode::Constant::String { ref value } => self.new_str(value.clone()),
             bytecode::Constant::Bytes { ref value } => self.new_bytes(value.clone()),
             bytecode::Constant::Boolean { ref value } => self.new_bool(value.clone()),
-            bytecode::Constant::Code { ref code } => self.new_code_object(code.clone()),
+            bytecode::Constant::Code { ref code } => self.new_code_object(*code.clone()),
             bytecode::Constant::Tuple { ref elements } => {
                 let elements = elements
                     .iter()

Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ pub enum Constant {`
`190`	`190`	`Boolean { value: bool },`
`191`	`191`	`String { value: String },`
`192`	`192`	`Bytes { value: Vec<u8> },`
`193`		`- Code { code: CodeObject },`
	`193`	`+ Code { code: Box<CodeObject> },`
`194`	`194`	`Tuple { elements: Vec<Constant> },`
`195`	`195`	`None,`
`196`	`196`	`}`
Original file line number	Diff line number	Diff line change
`@@ -716,8 +716,7 @@ impl Frame {`
`716`	`716`	`let obj = import_module(vm, current_path, module)?;`
`717`	`717`
`718`	`718`	`for (k, v) in obj.get_key_value_pairs().iter() {`
`719`		`- &self`
`720`		`- .scope`
	`719`	`+ self.scope`
`721`	`720`	`.locals`
`722`	`721`	`.set_item(&vm.ctx, &objstr::get_value(k), v.clone());`
`723`	`722`	`}`