From e30dc962b3079afeedbfe49bc6fa0a3f0a7b4780 Mon Sep 17 00:00:00 2001
From: Mark Olesen <Mark.Olesen@esi-group.com>
Date: Tue, 7 May 2019 20:39:20 +0200
Subject: [PATCH] STYLE: unnecessary read-check in ccm::writer

---
 src/conversion/ccm/writer/ccmWriterSolution.C | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/src/conversion/ccm/writer/ccmWriterSolution.C b/src/conversion/ccm/writer/ccmWriterSolution.C
index 32f7e01ee8a..3c979cff6ca 100644
--- a/src/conversion/ccm/writer/ccmWriterSolution.C
+++ b/src/conversion/ccm/writer/ccmWriterSolution.C
@@ -450,21 +450,14 @@ void Foam::ccm::writer::writeSolution
 
     forAllConstIters(objects, iter)
     {
-        word fieldName = (*iter()).name();
-        bool variableGood =
-        (
-            nameMapping.found(fieldName)
-         && (*iter()).typeHeaderOk<volScalarField>(false)
-        );
+        const word fieldName = (*iter()).name();
+        const word fieldType = (*iter()).headerClassName();
 
-        if (!variableGood)
+        if (!nameMapping.found(fieldName))
         {
-            // Only retain registered fields that are also readable
             continue;
         }
 
-        word fieldType = (*iter()).headerClassName();
-
         if (fieldType == volScalarField::typeName)
         {
             Info<< " " << fieldName << flush;
-- 
GitLab