From 9f9a67688e52d6ea4f3415d065b87bd9e395a9c4 Mon Sep 17 00:00:00 2001
From: Nav <nav@navnav.me>
Date: Sat, 17 Sep 2022 02:05:45 +0100
Subject: [PATCH] Tidying

---
 .../InsightWindow/InsightWindow.cpp           | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/Insight/UserInterfaces/InsightWindow/InsightWindow.cpp b/src/Insight/UserInterfaces/InsightWindow/InsightWindow.cpp
index 0e3efedd..bb84f88e 100644
--- a/src/Insight/UserInterfaces/InsightWindow/InsightWindow.cpp
+++ b/src/Insight/UserInterfaces/InsightWindow/InsightWindow.cpp
@@ -343,24 +343,26 @@ namespace Bloom
         }
 
         if (
-            variant.package == TargetPackage::DIP
-            || variant.package == TargetPackage::SOIC
-            || variant.package == TargetPackage::SSOP
+            (
+                variant.package == TargetPackage::DIP
+                || variant.package == TargetPackage::SOIC
+                || variant.package == TargetPackage::SSOP
+            )
+            && pinCount % 2 != 0
         ) {
             // All DIP, SOIC and SSOP variants must have a pin count that is a multiple of two
-            if (pinCount % 2 != 0) {
-                return false;
-            }
+            return false;
         }
 
-        if (variant.package == TargetPackage::QFP || variant.package == TargetPackage::QFN) {
+        if (
+            (variant.package == TargetPackage::QFP || variant.package == TargetPackage::QFN)
+            && (pinCount % 4 != 0 || pinCount <= 4)
+        ) {
             /*
              * All QFP and QFN variants must have a pin count that is a multiple of four. And there must be
              * more than one pin per side.
              */
-            if (pinCount % 4 != 0 || pinCount <= 4) {
-                return false;
-            }
+            return false;
         }
 
         return true;