Constant scalars: Don't flush double (#1315)

Paths were created in duplicate

Author: franzpoeschel

src/Mesh.cpp

@@ -243,25 +243,29 @@ void Mesh::flush_impl(
                 pCreate.path = name;
                 IOHandler()->enqueue(IOTask(this, pCreate));
                 for (auto &comp : *this)
+                {
                     comp.second.parent() = &this->writable();
+                    comp.second.flush(comp.first, flushParams);
+                }
             }
         }
-
-        if (scalar())
+        else
         {
-            for (auto &comp : *this)
+            if (scalar())
             {
-                comp.second.flush(name, flushParams);
-                writable().abstractFilePosition =
-                    comp.second.writable().abstractFilePosition;
+                for (auto &comp : *this)
+                {
+                    comp.second.flush(name, flushParams);
+                    writable().abstractFilePosition =
+                        comp.second.writable().abstractFilePosition;
+                }
+            }
+            else
+            {
+                for (auto &comp : *this)
+                    comp.second.flush(comp.first, flushParams);
             }
         }
-        else
-        {
-            for (auto &comp : *this)
-                comp.second.flush(comp.first, flushParams);
-        }
-
         flushAttributes(flushParams);
         break;
     }

src/Record.cpp

@@ -73,23 +73,29 @@ void Record::flush_impl(
                 pCreate.path = name;
                 IOHandler()->enqueue(IOTask(this, pCreate));
                 for (auto &comp : *this)
+                {
                     comp.second.parent() = getWritable(this);
+                    comp.second.flush(comp.first, flushParams);
+                }
             }
         }
-
-        if (scalar())
+        else
         {
-            for (auto &comp : *this)
+
+            if (scalar())
             {
-                comp.second.flush(name, flushParams);
-                writable().abstractFilePosition =
-                    comp.second.writable().abstractFilePosition;
+                for (auto &comp : *this)
+                {
+                    comp.second.flush(name, flushParams);
+                    writable().abstractFilePosition =
+                        comp.second.writable().abstractFilePosition;
+                }
+            }
+            else
+            {
+                for (auto &comp : *this)
+                    comp.second.flush(comp.first, flushParams);
             }
-        }
-        else
-        {
-            for (auto &comp : *this)
-                comp.second.flush(comp.first, flushParams);
         }
 
         flushAttributes(flushParams);