diff --git a/Framework/Kernel/src/ConfigService.cpp b/Framework/Kernel/src/ConfigService.cpp
index 0afc3bd879d63a1e07b1229b00a765f46281a93f..a5a6a6bf108abc66be154bff4a4f2e2bf554a31e 100644
--- a/Framework/Kernel/src/ConfigService.cpp
+++ b/Framework/Kernel/src/ConfigService.cpp
@@ -1779,8 +1779,10 @@ std::string ConfigServiceImpl::getFacilityFilename(const std::string &fName) {
 
   // If we are not updating the instrument definitions
   // update the iterator, this means we will skip the folder in HOME and
-  // look in the instrument folder in mantid install directory or mantid source code directory
-  if (updateInstrStr != "1" || updateInstrStr != "on" || updateInstrStr != "On") {
+  // look in the instrument folder in mantid install directory or mantid source
+  // code directory
+  if (updateInstrStr != "1" || updateInstrStr != "on" ||
+      updateInstrStr != "On") {
     instrDir++;
   }