remove superfluous commas

reviewable/pr1316/r4
Tamir Duberstein 2 weeks ago
parent 0d1193720b
commit a3aa387a2e
No known key found for this signature in database

@ -89,7 +89,7 @@ impl UProbe {
.ok_or(item.sig.span().error("expected `function` attribute"))?; .ok_or(item.sig.span().error("expected `function` attribute"))?;
match offset { match offset {
None => format!("{prefix}/{path}:{function}").into(), None => format!("{prefix}/{path}:{function}").into(),
Some(offset) => format!("{prefix}/{path}:{function}+{offset}",).into(), Some(offset) => format!("{prefix}/{path}:{function}+{offset}").into(),
} }
} }
}; };

@ -1662,7 +1662,7 @@ mod tests {
"kprobe/foo", "kprobe/foo",
&42u32.to_ne_bytes(), &42u32.to_ne_bytes(),
None, None,
),), )),
Err(ParseError::InvalidProgramCode) Err(ParseError::InvalidProgramCode)
); );
} }

@ -174,32 +174,32 @@ mod test {
#[test] #[test]
fn test_combine_flags() { fn test_combine_flags() {
assert_eq!( assert_eq!(
combine_flags(&to_vec("a b"), &to_vec("c d"),).join(" "), combine_flags(&to_vec("a b"), &to_vec("c d")).join(" "),
"a b c d", "a b c d",
); );
assert_eq!( assert_eq!(
combine_flags(&to_vec("a -- b"), &to_vec("a b"),).join(" "), combine_flags(&to_vec("a -- b"), &to_vec("a b")).join(" "),
"a a b -- b", "a a b -- b",
); );
assert_eq!( assert_eq!(
combine_flags(&to_vec("a -- b"), &to_vec("c d"),).join(" "), combine_flags(&to_vec("a -- b"), &to_vec("c d")).join(" "),
"a c d -- b", "a c d -- b",
); );
assert_eq!( assert_eq!(
combine_flags(&to_vec("a b"), &to_vec("c -- d"),).join(" "), combine_flags(&to_vec("a b"), &to_vec("c -- d")).join(" "),
"a b c -- d", "a b c -- d",
); );
assert_eq!( assert_eq!(
combine_flags(&to_vec("a -- b"), &to_vec("c -- d"),).join(" "), combine_flags(&to_vec("a -- b"), &to_vec("c -- d")).join(" "),
"a c -- b d", "a c -- b d",
); );
assert_eq!( assert_eq!(
combine_flags(&to_vec("a -- b"), &to_vec("-- c d"),).join(" "), combine_flags(&to_vec("a -- b"), &to_vec("-- c d")).join(" "),
"a -- b c d", "a -- b c d",
); );
} }

@ -251,7 +251,7 @@ fn create_probe_event(
.unwrap(); .unwrap();
let mut probe = OsString::new(); let mut probe = OsString::new();
write!(&mut probe, "{}:{}s/", probe_type_prefix, kind.pmu(),).unwrap(); write!(&mut probe, "{}:{}s/", probe_type_prefix, kind.pmu()).unwrap();
probe.push(&event_alias); probe.push(&event_alias);
probe.push(" "); probe.push(" ");
probe.push(fn_name); probe.push(fn_name);

@ -234,7 +234,7 @@ async fn ring_buf_async_with_drops() {
"seen={seen}, rejected={rejected}, dropped={dropped}, total={total}, max_seen={max_seen}, \ "seen={seen}, rejected={rejected}, dropped={dropped}, total={total}, max_seen={max_seen}, \
max_rejected={max_rejected}, max_dropped={max_dropped}", max_rejected={max_rejected}, max_dropped={max_dropped}",
); );
assert_eq!(seen + rejected + dropped, total, "{facts}",); assert_eq!(seen + rejected + dropped, total, "{facts}");
assert!( assert!(
(0u64..=max_dropped).contains(&dropped), (0u64..=max_dropped).contains(&dropped),
"dropped={dropped} not in 0..={max_dropped}; {facts}", "dropped={dropped} not in 0..={max_dropped}; {facts}",

Loading…
Cancel
Save