When returning an error, use "return" instead of "?"

Instead of

    Err(...)?;

which is correct but not idiomatic, write this:

    return Err(...);

This was found by clippy lintt try_err:
https://rust-lang.github.io/rust-clippy/master/index.html#try_err

Sponsored-by: author

1 files changed, 2 insertions, 2 deletions

diff --git a/sq/src/sq.rs b/sq/src/sq.rs
index fb6567f5..1c692ec0 100644
--- a/sq/src/sq.rs
+++ b/sq/src/sq.rs
@@ -137,8 +137,8 @@ fn load_keys<'a, I>(files: I) -> openpgp::Result<Vec<Cert>>
         let cert = Cert::from_file(f)
             .context(format!("Failed to load key from file {:?}", f))?;
         if ! cert.is_tsk() {
-            Err(anyhow::anyhow!(
-                "Cert in file {:?} does not contain secret keys", f))?;
+            return Err(anyhow::anyhow!(
+                "Cert in file {:?} does not contain secret keys", f));
         }
         certs.push(cert);
     }