Remove unnecessary `to_{owned,string}()` calls (#7367) · RustPython/RustPython@fc1c278 · GitHub
Skip to content

Commit fc1c278

Browse files
authored
Remove unnecessary to_{owned,string}() calls (#7367)
1 parent a27d812 commit fc1c278

71 files changed

Lines changed: 411 additions & 552 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

crates/stdlib/src/_asyncio.rs

Lines changed: 22 additions & 38 deletions

crates/stdlib/src/_remote_debugging.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ mod _remote_debugging {
9898
type Args = FuncArgs;
9999

100100
fn py_new(_cls: &Py<PyType>, _args: Self::Args, vm: &VirtualMachine) -> PyResult<Self> {
101-
Err(vm.new_not_implemented_error("_remote_debugging is not available".to_owned()))
101+
Err(vm.new_not_implemented_error("_remote_debugging is not available"))
102102
}
103103
}
104104

crates/stdlib/src/_sqlite3.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1512,9 +1512,9 @@ mod _sqlite3 {
15121512
let _ = unsafe { self.isolation_level.swap(value) };
15131513
Ok(())
15141514
}
1515-
PySetterValue::Delete => Err(vm.new_attribute_error(
1516-
"'isolation_level' attribute cannot be deleted".to_owned(),
1517-
)),
1515+
PySetterValue::Delete => {
1516+
Err(vm.new_attribute_error("'isolation_level' attribute cannot be deleted"))
1517+
}
15181518
}
15191519
}
15201520

crates/stdlib/src/faulthandler.rs

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -463,7 +463,7 @@ mod decl {
463463

464464
// Install signal handlers
465465
if !faulthandler_enable_internal() {
466-
return Err(vm.new_runtime_error("Failed to enable faulthandler".to_owned()));
466+
return Err(vm.new_runtime_error("Failed to enable faulthandler"));
467467
}
468468

469469
Ok(())
@@ -802,19 +802,15 @@ mod decl {
802802
// Check if it's an integer (file descriptor)
803803
if let Ok(fd) = f.try_to_value::<i32>(vm) {
804804
if fd < 0 {
805-
return Err(
806-
vm.new_value_error("file is not a valid file descriptor".to_owned())
807-
);
805+
return Err(vm.new_value_error("file is not a valid file descriptor"));
808806
}
809807
return Ok(fd);
810808
}
811809
// Try to get fileno() from file object
812810
let fileno = vm.call_method(&f, "fileno", ())?;
813811
let fd: i32 = fileno.try_to_value(vm)?;
814812
if fd < 0 {
815-
return Err(
816-
vm.new_value_error("file is not a valid file descriptor".to_owned())
817-
);
813+
return Err(vm.new_value_error("file is not a valid file descriptor"));
818814
}
819815
// Try to flush the file
820816
let _ = vm.call_method(&f, "flush", ());
@@ -824,7 +820,7 @@ mod decl {
824820
// file=None or file not passed: fall back to sys.stderr
825821
let stderr = vm.sys_module.get_attr("stderr", vm)?;
826822
if vm.is_none(&stderr) {
827-
return Err(vm.new_runtime_error("sys.stderr is None".to_owned()));
823+
return Err(vm.new_runtime_error("sys.stderr is None"));
828824
}
829825
let fileno = vm.call_method(&stderr, "fileno", ())?;
830826
let fd: i32 = fileno.try_to_value(vm)?;
@@ -912,15 +908,15 @@ mod decl {
912908
let timeout: f64 = args.timeout.into_float();
913909

914910
if timeout <= 0.0 {
915-
return Err(vm.new_value_error("timeout must be greater than 0".to_owned()));
911+
return Err(vm.new_value_error("timeout must be greater than 0"));
916912
}
917913

918914
let fd = get_fd_from_file_opt(args.file, vm)?;
919915

920916
// Convert timeout to microseconds
921917
let timeout_us = (timeout * 1_000_000.0) as u64;
922918
if timeout_us == 0 {
923-
return Err(vm.new_value_error("timeout must be greater than 0".to_owned()));
919+
return Err(vm.new_value_error("timeout must be greater than 0"));
924920
}
925921

926922
let header = format_timeout(timeout_us);
@@ -1098,7 +1094,7 @@ mod decl {
10981094

10991095
// Check if signal is in valid range
11001096
if !(1..64).contains(&signum) {
1101-
return Err(vm.new_value_error("signal number out of range".to_owned()));
1097+
return Err(vm.new_value_error("signal number out of range"));
11021098
}
11031099

11041100
Ok(())

crates/stdlib/src/hashlib.rs

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -210,8 +210,7 @@ pub mod _hashlib {
210210
(Some(_), Some(_)) => Err(vm.new_type_error(
211211
"'data' and 'string' are mutually exclusive \
212212
and support for 'string' keyword parameter \
213-
is slated for removal in a future version."
214-
.to_owned(),
213+
is slated for removal in a future version.",
215214
)),
216215
}
217216
}
@@ -306,7 +305,7 @@ pub mod _hashlib {
306305
impl PyHmac {
307306
#[pyslot]
308307
fn slot_new(_cls: PyTypeRef, _args: FuncArgs, vm: &VirtualMachine) -> PyResult {
309-
Err(vm.new_type_error("cannot create '_hashlib.HMAC' instances".to_owned()))
308+
Err(vm.new_type_error("cannot create '_hashlib.HMAC' instances"))
310309
}
311310

312311
#[pygetset]
@@ -758,9 +757,10 @@ pub mod _hashlib {
758757

759758
#[pyfunction]
760759
fn hmac_new(args: NewHMACHashArgs, vm: &VirtualMachine) -> PyResult<PyHmac> {
761-
let digestmod = args.digestmod.into_option().ok_or_else(|| {
762-
vm.new_type_error("Missing required parameter 'digestmod'.".to_owned())
763-
})?;
760+
let digestmod = args
761+
.digestmod
762+
.into_option()
763+
.ok_or_else(|| vm.new_type_error("Missing required parameter 'digestmod'."))?;
764764
let name = resolve_digestmod(&digestmod, vm)?;
765765

766766
let key_buf = args.key.borrow_buf();
@@ -833,10 +833,10 @@ pub mod _hashlib {
833833
let name = args.hash_name.as_str().to_lowercase();
834834

835835
if args.iterations < 1 {
836-
return Err(vm.new_value_error("iteration value must be greater than 0.".to_owned()));
836+
return Err(vm.new_value_error("iteration value must be greater than 0."));
837837
}
838838
let rounds = u32::try_from(args.iterations)
839-
.map_err(|_| vm.new_overflow_error("iteration value is too great.".to_owned()))?;
839+
.map_err(|_| vm.new_overflow_error("iteration value is too great."))?;
840840

841841
let dklen: usize = match args.dklen.into_option() {
842842
Some(obj) if vm.is_none(&obj) => {
@@ -845,10 +845,10 @@ pub mod _hashlib {
845845
Some(obj) => {
846846
let len: i64 = obj.try_into_value(vm)?;
847847
if len < 1 {
848-
return Err(vm.new_value_error("key length must be greater than 0.".to_owned()));
848+
return Err(vm.new_value_error("key length must be greater than 0."));
849849
}
850850
usize::try_from(len)
851-
.map_err(|_| vm.new_overflow_error("key length is too great.".to_owned()))?
851+
.map_err(|_| vm.new_overflow_error("key length is too great."))?
852852
}
853853
None => hash_digest_size(&name).ok_or_else(|| unsupported_hash(&name, vm))?,
854854
};

crates/stdlib/src/math.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,13 +102,13 @@ mod math {
102102
)));
103103
}
104104
if b == 1.0 {
105-
return Err(vm.new_value_error("math domain error".to_owned()));
105+
return Err(vm.new_value_error("math domain error"));
106106
}
107107
}
108108
// Handle BigInt specially for large values (only for actual int type, not float)
109109
if let Some(i) = x.downcast_ref::<PyInt>() {
110110
return pymath::math::log_bigint(i.as_bigint(), base).map_err(|err| match err {
111-
pymath::Error::EDOM => vm.new_value_error("expected a positive input".to_owned()),
111+
pymath::Error::EDOM => vm.new_value_error("expected a positive input"),
112112
_ => pymath_exception(err, vm),
113113
});
114114
}
@@ -132,7 +132,7 @@ mod math {
132132
// Handle BigInt specially for large values (only for actual int type, not float)
133133
if let Some(i) = x.downcast_ref::<PyInt>() {
134134
return pymath::math::log2_bigint(i.as_bigint()).map_err(|err| match err {
135-
pymath::Error::EDOM => vm.new_value_error("expected a positive input".to_owned()),
135+
pymath::Error::EDOM => vm.new_value_error("expected a positive input"),
136136
_ => pymath_exception(err, vm),
137137
});
138138
}
@@ -151,7 +151,7 @@ mod math {
151151
// Handle BigInt specially for large values (only for actual int type, not float)
152152
if let Some(i) = x.downcast_ref::<PyInt>() {
153153
return pymath::math::log10_bigint(i.as_bigint()).map_err(|err| match err {
154-
pymath::Error::EDOM => vm.new_value_error("expected a positive input".to_owned()),
154+
pymath::Error::EDOM => vm.new_value_error("expected a positive input"),
155155
_ => pymath_exception(err, vm),
156156
});
157157
}

crates/stdlib/src/mmap.rs

Lines changed: 4 additions & 6 deletions

0 commit comments

Comments
 (0)