From 686a8133a475b6c69f56c0dde69cb827ea0742c3 Mon Sep 17 00:00:00 2001
From: Matthew D Jones <matthew.d.jones@tessella.com>
Date: Fri, 28 Jul 2017 16:01:33 +0100
Subject: [PATCH] Re #20116 simplify comment
---
.../src/Instrument/InstrumentDefinitionParser.cpp | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/Framework/Geometry/src/Instrument/InstrumentDefinitionParser.cpp b/Framework/Geometry/src/Instrument/InstrumentDefinitionParser.cpp
index 4f7f555ffe7..755831cb682 100644
--- a/Framework/Geometry/src/Instrument/InstrumentDefinitionParser.cpp
+++ b/Framework/Geometry/src/Instrument/InstrumentDefinitionParser.cpp
@@ -294,11 +294,10 @@ InstrumentDefinitionParser::parseXML(Kernel::ProgressBase *progressReporter) {
checkComponentContainsLocationElement(pElem, filename);
- // Loop through all <location> and <locations> elements of this component
- // by looping all the child nodes and then see if any of these nodes are
- // either <location> or <locations> elements. Done this way order these
- // locations are processed is the order they are listed in the IDF. The
- // latter needed to get detector IDs assigned as expected
+ // Loop through all children of this component and see if any
+ // are a <location> or <locations>. Done this way, the
+ // order they are processed is the order they are listed in the
+ // IDF. This is necessary to match the order of the detector IDs.
for (Node *pNode = pElem->firstChild(); pNode != nullptr;
pNode = pNode->nextSibling()) {
auto pChildElem = dynamic_cast<Element *>(pNode);
--
GitLab