Bugfix: skip skipped compoments

Author: franzpoeschel

src/Mesh.cpp

@@ -632,6 +632,7 @@ void Mesh::read()
                           << "' and will skip it due to read error:\n"
                           << err.what() << std::endl;
                 map.forget(component);
+                continue;
             }
             homogenizeExtents.check_extent(*this, rc);
         }
@@ -660,6 +661,7 @@ void Mesh::read()
                           << "' and will skip it due to read error:\n"
                           << err.what() << std::endl;
                 map.forget(component);
+                continue;
             }
             homogenizeExtents.check_extent(*this, rc);
         }

src/ParticleSpecies.cpp

@@ -98,6 +98,7 @@ void ParticleSpecies::read()
                           << err.what() << std::endl;
 
                 map.forget(record_name);
+                continue;
             }
             homogenizeExtents.merge(*this, std::move(recordExtents));
         }
@@ -143,6 +144,7 @@ void ParticleSpecies::read()
             map.forget(record_name);
             //(*this)[record_name].erase(RecordComponent::SCALAR);
             // this->erase(record_name);
+            continue;
         }
         homogenizeExtents.merge(*this, std::move(recordExtents));
     }

src/Record.cpp

@@ -117,18 +117,22 @@ auto Record::read() -> internal::HomogenizeExtents
     };
     if (scalar())
     {
-        /* using operator[] will incorrectly update parent */
-        try
-        {
-            T_RecordComponent::read(/* require_unit_si = */ true);
-        }
-        catch (error::ReadError const &err)
-        {
-            std::cerr << "Cannot read scalar record component and will skip it "
-                         "due to read error:\n"
-                      << err.what() << std::endl;
-        }
-        check_extent(*this);
+        [&]() {
+            /* using operator[] will incorrectly update parent */
+            try
+            {
+                T_RecordComponent::read(/* require_unit_si = */ true);
+            }
+            catch (error::ReadError const &err)
+            {
+                std::cerr
+                    << "Cannot read scalar record component and will skip it "
+                       "due to read error:\n"
+                    << err.what() << std::endl;
+                return; // from lambda
+            }
+            check_extent(*this);
+        }();
     }
     else
     {
@@ -153,6 +157,7 @@ auto Record::read() -> internal::HomogenizeExtents
                           << "' and will skip it due to read error:\n"
                           << err.what() << std::endl;
                 this->container().erase(component);
+                continue;
             }
             check_extent(rc);
         }
@@ -181,6 +186,7 @@ auto Record::read() -> internal::HomogenizeExtents
                           << "' and will skip it due to read error:\n"
                           << err.what() << std::endl;
                 this->container().erase(component);
+                continue;
             }
             check_extent(rc);
         }