Fix this..

Author: franzpoeschel

include/openPMD/backend/Attributable.hpp

@@ -176,6 +176,12 @@ namespace internal
          * Attributable::setDirtyRecursive().
          */
         bool dirtyRecursive = true;
+
+        /**
+         * If frontend_parent is not null, then this is a key such that:
+         * &(*frontend_parent)[key] == this
+         */
+        std::string ownKeyWithinParent;
     };
 
     template <typename, typename>

include/openPMD/backend/ContainerImpl.tpp

@@ -144,11 +144,11 @@ auto Container<T, T_key, T_container>::operator[](key_type const &key)
         auto &ret = container().insert({key, std::move(t)}).first->second;
         if constexpr (std::is_same_v<T_key, std::string>)
         {
-            ret.writable().ownKeyWithinParent = key;
+            ret.m_attri->ownKeyWithinParent = key;
         }
         else
         {
-            ret.writable().ownKeyWithinParent = std::to_string(key);
+            ret.m_attri->ownKeyWithinParent = std::to_string(key);
         }
         traits::GenerationPolicy<T> gen;
         gen(ret, this);
@@ -176,12 +176,11 @@ auto Container<T, T_key, T_container>::operator[](key_type &&key)
         auto &ret = container().insert({key, std::move(t)}).first->second;
         if constexpr (std::is_same_v<T_key, std::string>)
         {
-            ret.writable().ownKeyWithinParent = std::move(key);
+            ret.m_attri->ownKeyWithinParent = std::move(key);
         }
         else
         {
-            ret.writable().ownKeyWithinParent =
-                std::to_string(std::move(key));
+            ret.m_attri->ownKeyWithinParent = std::to_string(std::move(key));
         }
         traits::GenerationPolicy<T> gen;
         gen(ret, this);

include/openPMD/backend/Writable.hpp

@@ -166,11 +166,6 @@ OPENPMD_private
     internal::AttributableData *attributable = nullptr;
     Writable *parent = nullptr;
 
-    /**
-     * If parent is not null, then this is a key such that:
-     * &(*parent)[key] == this
-     */
-    std::string ownKeyWithinParent;
     /**
      * @brief Whether a Writable has been written to the backend.
      *

src/Iteration.cpp

@@ -54,8 +54,8 @@ Iteration::Iteration() : Attributable(NoInit())
     setTime(static_cast<double>(0));
     setDt(static_cast<double>(1));
     setTimeUnitSI(1);
-    meshes.writable().ownKeyWithinParent = "meshes";
-    particles.writable().ownKeyWithinParent = "particles";
+    meshes.m_attri->ownKeyWithinParent = "meshes";
+    particles.m_attri->ownKeyWithinParent = "particles";
 }
 
 template <typename T>

src/ParticleSpecies.cpp

@@ -30,7 +30,7 @@ namespace openPMD
 {
 ParticleSpecies::ParticleSpecies()
 {
-    particlePatches.writable().ownKeyWithinParent = "particlePatches";
+    particlePatches.m_attri->ownKeyWithinParent = "particlePatches";
 }
 
 void ParticleSpecies::read()

src/Series.cpp

@@ -1084,7 +1084,7 @@ void Series::initSeries(
     }
 
     series.iterations.linkHierarchy(*this);
-    series.iterations.writable().ownKeyWithinParent = "data";
+    series.iterations.m_attri->ownKeyWithinParent = "data";
     series.m_rankTable.m_attributable.linkHierarchy(*this);
 
     series.m_name = input->name;
@@ -3687,18 +3687,19 @@ namespace debug
             };
             make_indent();
             auto const &w = attr.writable();
-            std::cout << w.ownKeyWithinParent << '\t' << attr.m_attri.get()
-                      << " -> " << &attr.writable() << '\n';
+            std::cout << attr.m_attri->ownKeyWithinParent << '\t'
+                      << attr.m_attri.get() << " -> " << &attr.writable()
+                      << '\n';
             make_indent();
             std::cout << "Self:\t" << attr.m_attri->dirtySelf
                       << "\tRec: " << attr.m_attri->dirtyRecursive << '\n';
             std::cout << '\n';
             graph << "{rank = same; ";
             graph << "_" << attr.m_attri.get() << "[color=green, label = \"A "
-                  << attr.m_attri.get() << " '" << w.ownKeyWithinParent
-                  << "'\"]; ";
+                  << attr.m_attri.get() << " '"
+                  << attr.m_attri->ownKeyWithinParent << "'\"]; ";
             graph << "_" << &w << "[color=blue, label = \"W " << &w << " '"
-                  << w.ownKeyWithinParent << "'\"]; ";
+                  << attr.m_attri->ownKeyWithinParent << "'\"]; ";
             graph << "}\n";
             graph << "_" << &w << " -> _" << attr.m_attri.get()
                   << "[dir=none];\n";

src/backend/Attributable.cpp

@@ -236,19 +236,19 @@ std::string Attributable::MyPath::openPMDPath() const
 auto Attributable::myPath() const -> MyPath
 {
     MyPath res;
-    Writable const *findSeries = &writable();
-    while (findSeries->parent)
+    internal::AttributableData *findSeries = m_attri.get();
+    while (findSeries->frontend_parent)
     {
         // we don't need to push_back the ownKeyWithinParent of the Series class
         // so it's alright that this loop doesn't ask the key of the last found
         // Writable
 
         res.group.push_back(findSeries->ownKeyWithinParent);
-        findSeries = findSeries->parent;
+        findSeries = findSeries->frontend_parent;
     }
     std::reverse(res.group.begin(), res.group.end());
-    auto &seriesData = auxiliary::deref_dynamic_cast<internal::SeriesData>(
-        findSeries->attributable);
+    auto &seriesData =
+        auxiliary::deref_dynamic_cast<internal::SeriesData>(findSeries);
     Series series;
     series.setData(
         std::shared_ptr<internal::SeriesData>{