Skip to content

Commit 7432621

Browse files
committed
Fix some more clippy warnings from rust nightly.
1 parent 30be872 commit 7432621

File tree

14 files changed

+33
-39
lines changed

14 files changed

+33
-39
lines changed

src/address.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ impl Address {
6060

6161
pub fn from_ip(ip: &IpAddr) -> Address {
6262
match ip {
63-
IpAddr::V4(v4) => Self::from_ipv4(&v4),
64-
IpAddr::V6(v6) => Self::from_ipv6(&v6),
63+
IpAddr::V4(v4) => Self::from_ipv4(v4),
64+
IpAddr::V6(v6) => Self::from_ipv6(v6),
6565
}
6666
}
6767
}

src/lib.rs

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ pub mod async_event_id;
77
pub use async_event_id::*;
88

99
mod buffer;
10-
pub use self::buffer::*;
10+
pub use buffer::*;
1111

1212
pub mod constants;
13-
pub use self::constants::*;
13+
pub use constants::*;
1414

1515
pub mod id;
1616
pub use id::*;
@@ -19,22 +19,20 @@ pub mod lifetime;
1919
pub use lifetime::*;
2020

2121
mod message;
22-
pub use self::message::*;
22+
pub use message::*;
2323

2424
pub mod monitor;
2525
pub use monitor::*;
2626

2727
pub mod nlas;
28-
pub use self::nlas::*;
28+
pub use nlas::*;
2929

3030
pub mod policy;
31-
pub use self::policy::*;
3231

3332
pub mod selector;
3433
pub use selector::*;
3534

3635
pub mod state;
37-
pub use state::*;
3836

3937
pub mod stats;
4038
pub use stats::*;

src/nlas/address_filter.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ impl AddressFilter {
8989
} else {
9090
self.splen = prefixlen;
9191
}
92-
self.family(&addr);
92+
self.family(addr);
9393
}
9494

9595
pub fn destination_prefix(&mut self, addr: &IpAddr, prefixlen: u8) {
@@ -100,6 +100,6 @@ impl AddressFilter {
100100
} else {
101101
self.dplen = prefixlen;
102102
}
103-
self.family(&addr);
103+
self.family(addr);
104104
}
105105
}

src/nlas/mod.rs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -179,14 +179,10 @@ impl Nla for XfrmAttrs {
179179
let mut it_tmpl = v.iter();
180180
let mut it_buf =
181181
buffer.chunks_exact_mut(XFRM_USER_TEMPLATE_LEN);
182-
loop {
183-
if let (Some(tmpl), Some(buf)) =
184-
(it_tmpl.next(), it_buf.next())
185-
{
186-
tmpl.emit(buf);
187-
} else {
188-
break;
189-
}
182+
while let (Some(tmpl), Some(buf)) =
183+
(it_tmpl.next(), it_buf.next())
184+
{
185+
tmpl.emit(buf);
190186
}
191187
}
192188
TfcPadding(ref v) => NativeEndian::write_u32(buffer, *v),

src/nlas/user_template.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ impl UserTemplate {
130130
/// should be the same type.
131131
pub fn source(&mut self, addr: &IpAddr) {
132132
self.saddr = Address::from_ip(addr);
133-
self.family(&addr);
133+
self.family(addr);
134134
}
135135

136136
/// Sets the destination address. Automatically sets the
@@ -139,7 +139,7 @@ impl UserTemplate {
139139
/// should be the same type.
140140
pub fn destination(&mut self, addr: &IpAddr) {
141141
self.id.daddr = Address::from_ip(addr);
142-
self.family(&addr);
142+
self.family(addr);
143143
}
144144

145145
/// Sets the transform protocol. Should be one of:

src/policy/default/message.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,8 @@
22

33
use anyhow::Context;
44

5-
use crate::{
6-
DefaultMessageBuffer, UserPolicyDefault, UserPolicyDefaultBuffer,
7-
POLICY_DEFAULT_HEADER_LEN,
8-
};
5+
use crate::policy::{DefaultMessageBuffer, POLICY_DEFAULT_HEADER_LEN};
6+
use crate::{UserPolicyDefault, UserPolicyDefaultBuffer};
97

108
use netlink_packet_utils::{traits::*, DecodeError};
119

src/policy/spdinfo/message.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
use anyhow::Context;
44

5-
use crate::{
5+
use crate::policy::{
66
GetSpdInfoMessageBuffer, NewSpdInfoMessageBuffer, SpdInfoAttrs,
77
POLICY_GET_SPD_INFO_HEADER_LEN, POLICY_NEW_SPD_INFO_HEADER_LEN,
88
};

src/selector.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ impl Selector {
124124
} else {
125125
self.prefixlen_s = prefixlen;
126126
}
127-
self.family(&addr);
127+
self.family(addr);
128128
}
129129

130130
pub fn destination_prefix(&mut self, addr: &IpAddr, prefixlen: u8) {
@@ -135,6 +135,6 @@ impl Selector {
135135
} else {
136136
self.prefixlen_d = prefixlen;
137137
}
138-
self.family(&addr);
138+
self.family(addr);
139139
}
140140
}

src/state/allocspi/message.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22

33
use anyhow::Context;
44

5-
use crate::{AllocSpiMessageBuffer, UserSpiInfo, UserSpiInfoBuffer, XfrmAttrs};
5+
use crate::state::AllocSpiMessageBuffer;
6+
use crate::{UserSpiInfo, UserSpiInfoBuffer, XfrmAttrs};
67

78
use netlink_packet_utils::{traits::*, DecodeError};
89

src/state/flush/message.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// SPDX-License-Identifier: MIT
22

3-
use crate::{state::FlushMessageBuffer, STATE_FLUSH_HEADER_LEN};
3+
use crate::state::{FlushMessageBuffer, STATE_FLUSH_HEADER_LEN};
44

55
use netlink_packet_utils::{traits::*, DecodeError};
66

0 commit comments

Comments
 (0)