From d2380859462c323d19e569138948b31e80c1fd9f Mon Sep 17 00:00:00 2001
From: Jake Smith <jake.smith@lexisnexisrisk.com>
Date: Tue, 5 Nov 2024 18:56:11 +0000
Subject: [PATCH] review change

Signed-off-by: Jake Smith <jake.smith@lexisnexisrisk.com>
---
 dali/ft/filecopy.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/dali/ft/filecopy.cpp b/dali/ft/filecopy.cpp
index dc25e283d68..641bf111e62 100644
--- a/dali/ft/filecopy.cpp
+++ b/dali/ft/filecopy.cpp
@@ -1597,13 +1597,12 @@ void FileSprayer::commonUpSlaves()
 
     // noCommon is defaulted to on for non-containerized (revisit!)
     bool noCommon = options->getPropBool(ANnocommon, !isContainerized());
-    if (isContainerized() && noCommon)
+    if (noCommon)
     {
+        if (!isContainerized())
+            return;
         IWARNLOG("Ignoring noCommon option in containerized mode");
-        noCommon = false;
     }
-    else if (noCommon)
-        return;
 
     //First work out which are the same slaves, and then map the partition.
     //Previously it was n^2 in partition, which is fine until you spray 100K files.