refactor: do not convert SQL arguments to String unnecessarily

This commit is contained in:
link2xt
2025-03-19 13:25:18 +00:00
committed by l
parent 73095bcaff
commit 8cb136ab9d

View File

@@ -847,7 +847,7 @@ impl Contact {
let row = transaction.query_row( let row = transaction.query_row(
"SELECT id, name, addr, origin, authname "SELECT id, name, addr, origin, authname
FROM contacts WHERE addr=? COLLATE NOCASE", FROM contacts WHERE addr=? COLLATE NOCASE",
[addr.to_string()], (addr,),
|row| { |row| {
let row_id: isize = row.get(0)?; let row_id: isize = row.get(0)?;
let row_name: String = row.get(1)?; let row_name: String = row.get(1)?;
@@ -961,16 +961,16 @@ impl Contact {
VALUES (?, ?, ?, ?);", VALUES (?, ?, ?, ?);",
( (
if update_name { if update_name {
name.to_string() &name
} else { } else {
"".to_string() ""
}, },
&addr, &addr,
origin, origin,
if update_authname { if update_authname {
name.to_string() &name
} else { } else {
"".to_string() ""
} }
), ),
)?; )?;