clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/apitest/at-otter.rs b/apitest/at-otter.rs
index 372997958e335a41be30977ac7c6cbfd5e9a7767..dbed7976e43c9c29aca7b757649438b9c2664754 100644 (file)
--- a/apitest/at-otter.rs
+++ b/apitest/at-otter.rs
@@ -198,7 +198,7 @@ impl Ctx {
   #[throws(Explode)]
   fn ssh_remote(&mut self) {
     let ds = self.ds().also(&[
-      ("config", PathResolveContext::default().resolve(&CONFIG)),
+      ("config", PathResolveContext::default().resolve(CONFIG)),
     ]);
     let mut command = ds.gss(
       "--ssh-command=@src@/apitest/mock-ssh \

diff --git a/apitest/atmain.rs b/apitest/atmain.rs
index 7b73a08b5df6bad3be72a06f4cc6fe2ed43d1a98..ad9cbb18602d0b67c1df4031432c92f113927c88 100644 (file)
--- a/apitest/atmain.rs
+++ b/apitest/atmain.rs
@@ -360,7 +360,7 @@ impl Session {
       |_session, gen      | (gen == exp).as_option(),
       | session, gen, k, v| {
         if let Some(pieces) = pieces.as_mut() {
-          update_update_pieces(&session.nick, pieces, k, v);
+          update_update_pieces(session.nick, pieces, k, v);
         }
         f(session,gen,k,v);
         None

diff --git a/cli/adhoc.rs b/cli/adhoc.rs
index 38e7e99863fbbfb8c7480a9bc11bdcec6ef73aea..cc5e44f940c40519328bdfff1fb0ce97d7948f6b 100644 (file)
--- a/cli/adhoc.rs
+++ b/cli/adhoc.rs
@@ -59,7 +59,7 @@ mod command_adhoc {
     cmds: Vec<String>,
   }
 
-  fn subargs<'ap,'a:'ap,'m:'ap>(
+  fn subargs<'ap,'a:'ap>(
     sa: &'a mut Args,
     ahf: AdhocFormat,
   ) -> ArgumentParser<'ap> {
@@ -108,7 +108,7 @@ mod alter_game_adhoc {
     insns: Vec<String>,
   }
 
-  fn subargs<'ap,'a:'ap,'m:'ap>(
+  fn subargs<'ap,'a:'ap>(
     sa: &'a mut Args,
     ahf: AdhocFormat,
   ) -> ArgumentParser<'ap> {

diff --git a/cli/forssh.rs b/cli/forssh.rs
index e19266b768b6bd5298e96790e471939ce62025ac..1f1494f787e4d619b82296c54117083937b7c2a0 100644 (file)
--- a/cli/forssh.rs
+++ b/cli/forssh.rs
@@ -150,7 +150,7 @@ mod set_ssh_keys {
     for (l, lno) in akf.lines().zip(1..) {
       let l = l.context("read KEYS-FILE")?;
       let l = l.trim();
-      if l.starts_with("#") || l == "" { continue }
+      if l.starts_with('#') || l == "" { continue }
       let l = AuthkeysLine(l.to_owned());
       let (pubdata, _comment) = l.parse()
         .with_context(|| format!("parse KEYS-FILE line {}", lno))?;

diff --git a/cli/otter.rs b/cli/otter.rs
index fe0bef98110de67dfda7d655bac4b99d59bdf62d..12cabe845dc87f2e4876ed23ee68163487ab84c0 100644 (file)
--- a/cli/otter.rs
+++ b/cli/otter.rs
@@ -304,8 +304,7 @@ fn main() {
     })?;
 
     let sc = inventory::iter::<Subcommand>.into_iter()
-      .filter(|Subcommand{verb:found,..}| found == &subcommand)
-      .next()
+      .find(|Subcommand{verb:found,..}| found == &subcommand)
       .unwrap_or_else(||{
         eprintln!("subcommand `{}' not recognised", &subcommand);
         exit(EXIT_USAGE);

diff --git a/cli/uselibs.rs b/cli/uselibs.rs
index c511d1b4c20710acfe981bb77e39071fe6ea130c..351f92d20b75b8f098e412726f015047db819b32 100644 (file)
--- a/cli/uselibs.rs
+++ b/cli/uselibs.rs
@@ -28,7 +28,7 @@ impl LibGlobArgs {
     self.lib.clone()
   }
   fn pat(&self) -> String {
-    self.pat.as_ref().map(Deref::deref)
+    self.pat.as_deref()
       .unwrap_or("*")
       .into()
   }
@@ -204,7 +204,7 @@ mod library_add {
       /// If returns None, has already maybe tried to take some space
       #[throws(AE)]
       fn place(&mut self, bbox: &Rect,
-               pieces: &Vec<MgmtGamePieceInfo>, ma: &MainOpts)
+               pieces: &[MgmtGamePieceInfo], ma: &MainOpts)
                -> Option<Pos> {
         let PosC{ coords: [w,h] } = (bbox.br() - bbox.tl())?;
 

diff --git a/clippy-options b/clippy-options
index b4cfc50f360636e79fe9e45ae397272cd0b28e65..34a9920c972c1ab7dde6328a9d90941e312a3bc1 100644 (file)
--- a/clippy-options
+++ b/clippy-options
@@ -37,3 +37,5 @@
 -A clippy::vec_init_then_push
 -A clippy::collapsible_if
 -A clippy::iter_nth_zero
+-A clippy::match_single_binding
+-A clippy::needless_update

diff --git a/jstest/jst-lower.rs b/jstest/jst-lower.rs
index b44d611efbb97b00b9d5c17fc21bd620951fc877..c03ae52ac8843781097b905f8d057058fa8ed043 100644 (file)
--- a/jstest/jst-lower.rs
+++ b/jstest/jst-lower.rs
@@ -362,7 +362,6 @@ impl TestsAccumulator {
     let pieces: IndexMap<Vpid,StartPiece> = pieces.into_iter().map(
       |StartPieceSpec { id, pinned, moveable, zupd }| {
         let zupd_d = (&zupd).into();
-        let id = id.try_into().unwrap();
         let zlevel = zupd.next(&mut zlast, &mut zlastg);
         (id, StartPiece { pinned, moveable, zlevel, zupd: zupd_d })
       }