summaryrefslogtreecommitdiffstats
path: root/net/src
diff options
context:
space:
mode:
authorLars Wirzenius <liw@liw.fi>2021-09-27 09:04:34 +0300
committerLars Wirzenius <liw@sequoia-pgp.org>2021-09-30 08:31:07 +0300
commit85362f33d81ebdc1cd355454be2494b9286e80be (patch)
tree6fba218d7f81565e0dd7f00883d8ab908e958af0 /net/src
parent9fdbbf44d55f99807b2d2736ed3a85b0517be8e7 (diff)
Avoid naming field setting it from variable of the same name
When creating a struct with a field foo, using a variable also named foo, it's not necessary to name the field explicitly. Thus, instead of: Self { foo: foo } use this: Self { foo } The shorter form is more idiomatic and thus less confusing to experienced Rust programmers. This was found by the clippy lint redundant_field_names: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names Sponsored-by: author
Diffstat (limited to 'net/src')
-rw-r--r--net/src/wkd.rs2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/src/wkd.rs b/net/src/wkd.rs
index dfe9a6e8..6662567a 100644
--- a/net/src/wkd.rs
+++ b/net/src/wkd.rs
@@ -128,7 +128,7 @@ impl Url {
let local_encoded = encode_local_part(&email.local_part);
let url = Url {
domain : email.domain,
- local_encoded : local_encoded,
+ local_encoded,
local_part : email.local_part,
};
Ok(url)