Join nested if statements with logical and into one statement

Instead of this:

    if foo {
        if bar {
           ...
        }
    }

do this:

    if foo && bar {
        ...
    }

Nesting statements implies a more complicated code structure than it
really is. Thus it's arguably simpler to write a combined condition by
joining the two conditions with a logical and operation.

Found by clippy lint collapsible_if:
https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if

1 files changed, 4 insertions, 6 deletions

diff --git a/ipc/src/core.rs b/ipc/src/core.rs
index b4dd928a..6712921b 100644
--- a/ipc/src/core.rs
+++ b/ipc/src/core.rs
@@ -179,12 +179,10 @@ impl Config {
             let tmp = tempfile::Builder::new().prefix("sequoia").tempdir()?;
             c.home = tmp.into_path();
             c.cleanup = true;
-        } else {
-            if home_not_set {
-                c.home = dirs::home_dir()
-                    .ok_or_else(|| anyhow::anyhow!("Failed to get users home directory"))?
-                    .join(".sequoia");
-            }
+        } else if home_not_set {
+            c.home = dirs::home_dir()
+                .ok_or_else(|| anyhow::anyhow!("Failed to get users home directory"))?
+            .join(".sequoia");
         }
         Ok(c)
     }