summaryrefslogtreecommitdiffstats
path: root/ffi
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 /ffi
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 'ffi')
-rw-r--r--ffi/src/core.rs2
1 files changed, 1 insertions, 1 deletions
diff --git a/ffi/src/core.rs b/ffi/src/core.rs
index d9bccd3b..147b9eba 100644
--- a/ffi/src/core.rs
+++ b/ffi/src/core.rs
@@ -52,7 +52,7 @@ pub struct Context {
impl Context {
fn new(c: sequoia_ipc::Context) -> Self {
- Context{c: c, e: ptr::null_mut()}
+ Context{c, e: ptr::null_mut()}
}
pub(crate) fn errp(&mut self) -> &mut *mut crate::error::Error {