Reviewer's suggestions

Author: franzpoeschel

include/openPMD/IO/ADIOS/ParallelADIOS1IOHandlerImpl.hpp

@@ -58,7 +58,8 @@ namespace openPMD
         void extendDataset(Writable*, Parameter< Operation::EXTEND_DATASET > const&) override;
         void openFile(Writable*, Parameter< Operation::OPEN_FILE > const&) override;
         void closeFile(Writable*, Parameter< Operation::CLOSE_FILE > const&) override;
-        void availableChunks(Writable*, Parameter< Operation::AVAILABLE_CHUNKS > &) override;void openPath(Writable*, Parameter< Operation::OPEN_PATH > const&) override;
+        void availableChunks(Writable*, Parameter< Operation::AVAILABLE_CHUNKS > &) override;
+        void openPath(Writable*, Parameter< Operation::OPEN_PATH > const&) override;
         void openDataset(Writable*, Parameter< Operation::OPEN_DATASET > &) override;
         void deleteFile(Writable*, Parameter< Operation::DELETE_FILE > const&) override;
         void deletePath(Writable*, Parameter< Operation::DELETE_PATH > const&) override;

include/openPMD/IO/AbstractIOHandlerImpl.hpp

@@ -130,15 +130,16 @@ class AbstractIOHandlerImpl
    */
   virtual void
   closeFile( Writable *, Parameter< Operation::CLOSE_FILE > const & ) = 0;
+
   /** Report chunks that are available for loading from the dataset represented
    *  by this writable.
    *
    * The resulting chunks should be stored into parameters.chunks.
    *
    */
   virtual void
-  availableChunks( Writable *, Parameter< Operation::AVAILABLE_CHUNKS > & )
-  {}
+  availableChunks( Writable *, Parameter< Operation::AVAILABLE_CHUNKS > & ) = 0;
+
   /** Create a new file in physical storage, possibly overriding an existing file.
    *
    * The operation should fail if m_handler->m_frontendAccess is Access::READ_ONLY.