Undo unnecessary whitespace changes

Author: franzpoeschel

include/openPMD/IO/ADIOS/ADIOS2IOHandler.hpp

@@ -1326,7 +1326,8 @@ namespace detail
          */
         bool finalized = false;
 
-        void configure_IO( ADIOS2IOHandlerImpl & impl );
+        void
+        configure_IO( ADIOS2IOHandlerImpl & impl );
 
         using AttributeLayout = ADIOS2IOHandlerImpl::AttributeLayout;
         inline AttributeLayout attributeLayout() const
@@ -1337,37 +1338,35 @@ namespace detail
         }
     };
 
-    } // namespace detail
+} // namespace detail
 #endif // openPMD_HAVE_ADIOS2
 
-    class ADIOS2IOHandler : public AbstractIOHandler
-    {
+class ADIOS2IOHandler : public AbstractIOHandler
+{
 #if openPMD_HAVE_ADIOS2
 
-        friend class ADIOS2IOHandlerImpl;
+    friend class ADIOS2IOHandlerImpl;
 
-    private:
-        ADIOS2IOHandlerImpl m_impl;
+private:
+    ADIOS2IOHandlerImpl m_impl;
 
-    public:
-        ~ADIOS2IOHandler() override
+public:
+    ~ADIOS2IOHandler( ) override
+    {
+        // we must not throw in a destructor
+        try
         {
-            // we must not throw in a destructor
-            try
-            {
-                this->flush();
-            }
-            catch( std::exception const & ex )
-            {
-                std::cerr << "[~ADIOS2IOHandler] An error occurred: "
-                          << ex.what() << std::endl;
-            }
-            catch( ... )
-            {
-                std::cerr << "[~ADIOS2IOHandler] An error occurred."
-                          << std::endl;
-            }
+            this->flush( );
         }
+        catch( std::exception const & ex )
+        {
+            std::cerr << "[~ADIOS2IOHandler] An error occurred: " << ex.what() << std::endl;
+        }
+        catch( ... )
+        {
+             std::cerr << "[~ADIOS2IOHandler] An error occurred." << std::endl;
+        }
+    }
 
 #else
 public:
@@ -1393,5 +1392,5 @@ namespace detail
     std::string backendName() const override { return "ADIOS2"; }
 
     std::future< void > flush() override;
-    }; // ADIOS2IOHandler
-    } // namespace openPMD
+}; // ADIOS2IOHandler
+} // namespace openPMD

src/IO/ADIOS/ADIOS2IOHandler.cpp

@@ -900,7 +900,7 @@ ADIOS2IOHandlerImpl::availableChunks(
 adios2::Mode
 ADIOS2IOHandlerImpl::adios2AccessMode( std::string const & fullPath )
 {
-    switch( m_handler->m_backendAccess )
+    switch ( m_handler->m_backendAccess )
     {
     case Access::CREATE:
         return adios2::Mode::Write;
@@ -2539,29 +2539,30 @@ namespace detail
             ADIOS2Defaults::str_usesstepsAttribute, 1 );
         switch( mode )
         {
-        case AdvanceMode::ENDSTEP: {
-            /*
-             * Advance mode write:
-             * Close the current step, defer opening the new step
-             * until one is actually needed:
-             * (1) The engine is accessed in BufferedActions::flush
-             * (2) A new step is opened before the currently active step
-             *     has seen an access. See the following lines: open the
-             *     step just to skip it again.
-             */
-            if( streamStatus == StreamStatus::OutsideOfStep )
+            case AdvanceMode::ENDSTEP:
             {
-                getEngine().BeginStep();
-            }
-            flush(
-                []( BufferedActions &, adios2::Engine & eng ) {
-                    eng.EndStep();
-                },
-                /* writeAttributes = */ true,
-                /* flushUnconditionally = */ true );
-            uncommittedAttributes.clear();
-            streamStatus = StreamStatus::OutsideOfStep;
-            return AdvanceStatus::OK;
+                /*
+                 * Advance mode write:
+                 * Close the current step, defer opening the new step
+                 * until one is actually needed:
+                 * (1) The engine is accessed in BufferedActions::flush
+                 * (2) A new step is opened before the currently active step
+                 *     has seen an access. See the following lines: open the
+                 *     step just to skip it again.
+                 */
+                if( streamStatus == StreamStatus::OutsideOfStep )
+                {
+                    getEngine().BeginStep();
+                }
+                flush(
+                    []( BufferedActions &, adios2::Engine & eng ) {
+                        eng.EndStep();
+                    },
+                    /* writeAttributes = */ true,
+                    /* flushUnconditionally = */ true );
+                uncommittedAttributes.clear();
+                streamStatus = StreamStatus::OutsideOfStep;
+                return AdvanceStatus::OK;
             }
             case AdvanceMode::BEGINSTEP:
             {
@@ -2604,7 +2605,7 @@ namespace detail
                 invalidateVariablesMap();
                 return res;
             }
-            }
+        }
         throw std::runtime_error(
             "Internal error: Advance mode should be explicitly"
             " chosen by the front-end." );