Windows fixes

openPMD · Feb 16, 2024 · b5e4217 · b5e4217
1 parent 993262f
commit b5e4217
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 4 deletions.
diff --git a/include/openPMD/snapshots/ContainerTraits.hpp b/include/openPMD/snapshots/ContainerTraits.hpp
@@ -14,7 +14,8 @@ class OpaqueSeriesIterator
     friend class Series;
     friend class StatefulSnapshotsContainer;
     friend class RandomAccessIteratorContainer;
-    using parent_t = AbstractSeriesIterator<OpaqueSeriesIterator>;
+    using parent_t =
+        AbstractSeriesIterator<OpaqueSeriesIterator, value_type_in>;
     // no shared_ptr since copied iterators should not share state
     std::unique_ptr<DynamicSeriesIterator<value_type_in>> m_internal_iterator;
 

diff --git a/include/openPMD/snapshots/RandomAccessIterator.hpp b/include/openPMD/snapshots/RandomAccessIterator.hpp
@@ -41,7 +41,9 @@ class RandomAccessIterator
 {
 private:
     friend class RandomAccessIteratorContainer;
-    using parent_t = AbstractSeriesIterator<RandomAccessIterator<iterator_t>>;
+    using parent_t = AbstractSeriesIterator<
+        RandomAccessIterator<iterator_t>,
+        detail::iterator_to_value_type<iterator_t>>;
 
     RandomAccessIterator(iterator_t it);
 

diff --git a/include/openPMD/snapshots/StatefulIterator.hpp b/include/openPMD/snapshots/StatefulIterator.hpp
@@ -148,7 +148,9 @@ class StatefulIterator
     auto get() -> SharedData &;
     auto get() const -> SharedData const &;
 
-    using parent_t = AbstractSeriesIterator<StatefulIterator>;
+    using parent_t = AbstractSeriesIterator<
+        StatefulIterator,
+        Container<Iteration, Iteration::IterationIndex_t>::value_type>;
 
 public:
     using value_type =

diff --git a/src/IO/ADIOS/ADIOS2IOHandler.cpp b/src/IO/ADIOS/ADIOS2IOHandler.cpp
@@ -1588,6 +1588,7 @@ detail::ADIOS2File &ADIOS2IOHandlerImpl::getFileData(
     {
         return *it->second;
     }
+    throw std::runtime_error("Unreachable!");
 }
 
 void ADIOS2IOHandlerImpl::dropFileData(InvalidatableFile const &file)

diff --git a/src/snapshots/StatefulIterator.cpp b/src/snapshots/StatefulIterator.cpp
@@ -535,7 +535,6 @@ std::optional<StatefulIterator *> StatefulIterator::loopBody()
                 try
                 {
                     iterations.at(current_iteration).open();
-                    [[fallthrough]];
                 }
                 catch (error::ReadError const &err)
                 {
@@ -545,6 +544,7 @@ std::optional<StatefulIterator *> StatefulIterator::loopBody()
                     option.value()->deactivateDeadIteration(current_iteration);
                     return std::nullopt;
                 }
+                [[fallthrough]];
             case internal::CloseStatus::Open:
                 return option;
             case internal::CloseStatus::ClosedInBackend: