Fix clippy issues for rust 1.67

DimitrisJim · DimitrisJim · commit d4b9ad863ff3 · 2023-01-30T21:40:07.000+02:00
diff --git a/stdlib/src/sqlite.rs b/stdlib/src/sqlite.rs
@@ -557,13 +557,13 @@ mod _sqlite {
                     context.result_exception(
                         vm,
                         exc,
-                        &format!("user-defined aggregate's '{}' method not defined\0", name),
+                        &format!("user-defined aggregate's '{name}' method not defined\0"),
                     )
                 } else {
                     context.result_exception(
                         vm,
                         exc,
-                        &format!("user-defined aggregate's '{}' method raised error\0", name),
+                        &format!("user-defined aggregate's '{name}' method raised error\0"),
                     )
                 }
             }
@@ -591,13 +591,13 @@ mod _sqlite {
                     context.result_exception(
                         vm,
                         exc,
-                        &format!("user-defined aggregate's '{}' method not defined\0", name),
+                        &format!("user-defined aggregate's '{name}' method not defined\0"),
                     )
                 } else {
                     context.result_exception(
                         vm,
                         exc,
-                        &format!("user-defined aggregate's '{}' method raised error\0", name),
+                        &format!("user-defined aggregate's '{name}' method raised error\0"),
                     )
                 }
             }
@@ -1739,8 +1739,7 @@ mod _sqlite {
                         }
                         _ => {
                             return Err(vm.new_not_implemented_error(format!(
-                                "unknown column type: {}",
-                                col_type
+                                "unknown column type: {col_type}"
                             )));
                         }
                     }
diff --git a/vm/src/builtins/function/jitfunc.rs b/vm/src/builtins/function/jitfunc.rs
@@ -58,7 +58,7 @@ fn get_jit_arg_type(dict: &PyDictRef, name: &str, vm: &VirtualMachine) -> PyResu
         }
     } else {
         Err(new_jit_error(
-            format!("argument {} needs annotation", name),
+            format!("argument {name} needs annotation"),
             vm,
         ))
     }
diff --git a/vm/src/format.rs b/vm/src/format.rs
@@ -18,18 +18,18 @@ impl IntoPyException for FormatSpecError {
                 vm.new_value_error("Invalid format specifier".to_owned())
             }
             FormatSpecError::UnspecifiedFormat(c1, c2) => {
-                let msg = format!("Cannot specify '{}' with '{}'.", c1, c2);
+                let msg = format!("Cannot specify '{c1}' with '{c2}'.");
                 vm.new_value_error(msg)
             }
             FormatSpecError::UnknownFormatCode(c, s) => {
-                let msg = format!("Unknown format code '{}' for object of type '{}'", c, s);
+                let msg = format!("Unknown format code '{c}' for object of type '{s}'");
                 vm.new_value_error(msg)
             }
             FormatSpecError::PrecisionNotAllowed => {
                 vm.new_value_error("Precision not allowed in integer format specifier".to_owned())
             }
             FormatSpecError::NotAllowed(s) => {
-                let msg = format!("{} not allowed with integer format specifier 'c'", s);
+                let msg = format!("{s} not allowed with integer format specifier 'c'");
                 vm.new_value_error(msg)
             }
             FormatSpecError::UnableToConvert => {
@@ -39,10 +39,7 @@ impl IntoPyException for FormatSpecError {
                 vm.new_overflow_error("%c arg not in range(0x110000)".to_owned())
             }
             FormatSpecError::NotImplemented(c, s) => {
-                let msg = format!(
-                    "Format code '{}' for object of type '{}' not implemented yet",
-                    c, s
-                );
+                let msg = format!("Format code '{c}' for object of type '{s}' not implemented yet");
                 vm.new_value_error(msg)
             }
         }
diff --git a/vm/src/stdlib/posix.rs b/vm/src/stdlib/posix.rs
@@ -571,10 +571,7 @@ pub mod module {
             let priority_type = priority_class.name();
             let priority = self.sched_priority.clone();
             let value = priority.downcast::<PyInt>().map_err(|_| {
-                vm.new_type_error(format!(
-                    "an integer is required (got type {})",
-                    priority_type
-                ))
+                vm.new_type_error(format!("an integer is required (got type {priority_type})"))
             })?;
             let sched_priority = value.try_to_primitive(vm)?;
             Ok(libc::sched_param { sched_priority })
@@ -1991,7 +1988,7 @@ pub mod module {
     #[pyfunction]
     fn getrandom(size: isize, flags: OptionalArg<u32>, vm: &VirtualMachine) -> PyResult<Vec<u8>> {
         let size = usize::try_from(size)
-            .map_err(|_| vm.new_os_error(format!("Invalid argument for size: {}", size)))?;
+            .map_err(|_| vm.new_os_error(format!("Invalid argument for size: {size}")))?;
         let mut buf = Vec::with_capacity(size);
         unsafe {
             let len = sys_getrandom(

Original file line number	Diff line number	Diff line change
`@@ -557,13 +557,13 @@ mod _sqlite {`
`557`	`557`	`context.result_exception(`
`558`	`558`	`vm,`
`559`	`559`	`exc,`
`560`		`- &format!("user-defined aggregate's '{}' method not defined\0", name),`
	`560`	`+ &format!("user-defined aggregate's '{name}' method not defined\0"),`
`561`	`561`	`)`
`562`	`562`	`} else {`
`563`	`563`	`context.result_exception(`
`564`	`564`	`vm,`
`565`	`565`	`exc,`
`566`		`- &format!("user-defined aggregate's '{}' method raised error\0", name),`
	`566`	`+ &format!("user-defined aggregate's '{name}' method raised error\0"),`
`567`	`567`	`)`
`568`	`568`	`}`
`569`	`569`	`}`
`@@ -591,13 +591,13 @@ mod _sqlite {`
`591`	`591`	`context.result_exception(`
`592`	`592`	`vm,`
`593`	`593`	`exc,`
`594`		`- &format!("user-defined aggregate's '{}' method not defined\0", name),`
	`594`	`+ &format!("user-defined aggregate's '{name}' method not defined\0"),`
`595`	`595`	`)`
`596`	`596`	`} else {`
`597`	`597`	`context.result_exception(`
`598`	`598`	`vm,`
`599`	`599`	`exc,`
`600`		`- &format!("user-defined aggregate's '{}' method raised error\0", name),`
	`600`	`+ &format!("user-defined aggregate's '{name}' method raised error\0"),`
`601`	`601`	`)`
`602`	`602`	`}`
`603`	`603`	`}`
`@@ -1739,8 +1739,7 @@ mod _sqlite {`
`1739`	`1739`	`}`
`1740`	`1740`	`_ => {`
`1741`	`1741`	`return Err(vm.new_not_implemented_error(format!(`
`1742`		`- "unknown column type: {}",`
`1743`		`- col_type`
	`1742`	`+ "unknown column type: {col_type}"`
`1744`	`1743`	`)));`
`1745`	`1744`	`}`
`1746`	`1745`	`}`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ fn get_jit_arg_type(dict: &PyDictRef, name: &str, vm: &VirtualMachine) -> PyResu`
`58`	`58`	`}`
`59`	`59`	`} else {`
`60`	`60`	`Err(new_jit_error(`
`61`		`- format!("argument {} needs annotation", name),`
	`61`	`+ format!("argument {name} needs annotation"),`
`62`	`62`	`vm,`
`63`	`63`	`))`
`64`	`64`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,18 +18,18 @@ impl IntoPyException for FormatSpecError {`
`18`	`18`	`vm.new_value_error("Invalid format specifier".to_owned())`
`19`	`19`	`}`
`20`	`20`	`FormatSpecError::UnspecifiedFormat(c1, c2) => {`
`21`		`- let msg = format!("Cannot specify '{}' with '{}'.", c1, c2);`
	`21`	`+ let msg = format!("Cannot specify '{c1}' with '{c2}'.");`
`22`	`22`	`vm.new_value_error(msg)`
`23`	`23`	`}`
`24`	`24`	`FormatSpecError::UnknownFormatCode(c, s) => {`
`25`		`- let msg = format!("Unknown format code '{}' for object of type '{}'", c, s);`
	`25`	`+ let msg = format!("Unknown format code '{c}' for object of type '{s}'");`
`26`	`26`	`vm.new_value_error(msg)`
`27`	`27`	`}`
`28`	`28`	`FormatSpecError::PrecisionNotAllowed => {`
`29`	`29`	`vm.new_value_error("Precision not allowed in integer format specifier".to_owned())`
`30`	`30`	`}`
`31`	`31`	`FormatSpecError::NotAllowed(s) => {`
`32`		`- let msg = format!("{} not allowed with integer format specifier 'c'", s);`
	`32`	`+ let msg = format!("{s} not allowed with integer format specifier 'c'");`
`33`	`33`	`vm.new_value_error(msg)`
`34`	`34`	`}`
`35`	`35`	`FormatSpecError::UnableToConvert => {`
`@@ -39,10 +39,7 @@ impl IntoPyException for FormatSpecError {`
`39`	`39`	`vm.new_overflow_error("%c arg not in range(0x110000)".to_owned())`
`40`	`40`	`}`
`41`	`41`	`FormatSpecError::NotImplemented(c, s) => {`
`42`		`- let msg = format!(`
`43`		`- "Format code '{}' for object of type '{}' not implemented yet",`
`44`		`- c, s`
`45`		`- );`
	`42`	`+ let msg = format!("Format code '{c}' for object of type '{s}' not implemented yet");`
`46`	`43`	`vm.new_value_error(msg)`
`47`	`44`	`}`
`48`	`45`	`}`