From 28967950b65198737aa9b2a011ac9a4b7d54f180 Mon Sep 17 00:00:00 2001
From: Simon Praetorius <simon.praetorius@tu-dresden.de>
Date: Sun, 26 Apr 2020 18:00:05 +0200
Subject: [PATCH] replace Dune::Std::optional with std::optional

---
 dune/vtk/filewriter.hh                             |  4 ++--
 dune/vtk/gridcreators/lagrangegridcreator.hh       |  6 +++---
 dune/vtk/pvdwriter.hh                              |  7 +++----
 dune/vtk/pvdwriter.impl.hh                         |  4 ++--
 dune/vtk/vtkfunction.hh                            | 10 +++++-----
 dune/vtk/vtktimeserieswriter.hh                    |  7 +++----
 dune/vtk/vtktimeserieswriter.impl.hh               |  4 ++--
 dune/vtk/vtkwriterinterface.hh                     |  8 ++++----
 dune/vtk/vtkwriterinterface.impl.hh                |  6 +++---
 dune/vtk/writers/vtkrectilineargridwriter.hh       |  2 +-
 dune/vtk/writers/vtkrectilineargridwriter.impl.hh  |  2 +-
 dune/vtk/writers/vtkunstructuredgridwriter.hh      |  4 ++--
 dune/vtk/writers/vtkunstructuredgridwriter.impl.hh |  4 ++--
 13 files changed, 33 insertions(+), 35 deletions(-)

diff --git a/dune/vtk/filewriter.hh b/dune/vtk/filewriter.hh
index ecddbd2..803fe25 100644
--- a/dune/vtk/filewriter.hh
+++ b/dune/vtk/filewriter.hh
@@ -1,8 +1,8 @@
 #pragma once
 
+#include <optional>
 #include <string>
 
-#include <dune/common/std/optional.hh>
 #include <dune/vtk/forward.hh>
 
 namespace Dune
@@ -14,7 +14,7 @@ namespace Dune
     virtual ~FileWriter () = default;
 
     /// Write to file given by `filename` and (optionally) store additional data in `dataDir`
-    virtual std::string write (std::string const& filename, Std::optional<std::string> dataDir = {}) const = 0;
+    virtual std::string write (std::string const& filename, std::optional<std::string> dataDir = {}) const = 0;
   };
 
 } // end namespace Dune
diff --git a/dune/vtk/gridcreators/lagrangegridcreator.hh b/dune/vtk/gridcreators/lagrangegridcreator.hh
index fa2a585..c39750f 100644
--- a/dune/vtk/gridcreators/lagrangegridcreator.hh
+++ b/dune/vtk/gridcreators/lagrangegridcreator.hh
@@ -3,11 +3,11 @@
 #include <cassert>
 #include <cstdint>
 #include <limits>
+#include <optional>
 #include <vector>
 
 #include <dune/common/exceptions.hh>
 #include <dune/common/hybridutilities.hh>
-#include <dune/common/std/optional.hh>
 #include <dune/geometry/utility/typefromvertexcount.hh>
 #include <dune/localfunctions/lagrange.hh>
 #include <dune/grid/common/gridfactory.hh>
@@ -297,7 +297,7 @@ namespace Dune
   private:
     LocalFE localFE_;
     std::vector<GlobalCoordinate> localNodes_;
-    Std::optional<LocalGeometry> localGeometry_;
+    std::optional<LocalGeometry> localGeometry_;
 
     mutable std::vector<typename LocalBasisTraits::RangeType> shapeValues_;
   };
@@ -343,7 +343,7 @@ namespace Dune
     LagrangeGridCreator const* gridCreator_;
 
     LocalContext localContext_;
-    Std::optional<LocalParametrization> localParametrization_;
+    std::optional<LocalParametrization> localParametrization_;
   };
 
 } // end namespace Dune
diff --git a/dune/vtk/pvdwriter.hh b/dune/vtk/pvdwriter.hh
index 38fef8f..7b256cc 100644
--- a/dune/vtk/pvdwriter.hh
+++ b/dune/vtk/pvdwriter.hh
@@ -1,12 +1,11 @@
 #pragma once
 
 #include <iosfwd>
+#include <optional>
 #include <string>
 #include <vector>
 #include <tuple>
 
-#include <dune/common/std/optional.hh>
-
 #include <dune/vtk/vtktypes.hh>
 #include <dune/vtk/filewriter.hh>
 #include <dune/vtk/forward.hh>
@@ -42,7 +41,7 @@ namespace Dune
      * \param dir   Specifies where to write the timestep files.
      * \param writeCollection  Create a collection .pvd file directly
      **/
-    void writeTimestep (double time, std::string const& fn, Std::optional<std::string> dir = {},
+    void writeTimestep (double time, std::string const& fn, std::optional<std::string> dir = {},
                         bool writeCollection = true) const;
 
     /// \brief Writes collection of timesteps to .pvd file.
@@ -52,7 +51,7 @@ namespace Dune
      * \param dir (Ignored) Timestep files are already written and their filenames are
      *            stored in \ref timesteps_.
      **/
-    virtual std::string write (std::string const& fn, Std::optional<std::string> dir = {}) const override;
+    virtual std::string write (std::string const& fn, std::optional<std::string> dir = {}) const override;
 
     /// Attach point data to the writer, \see VtkFunction for possible arguments
     template <class Function, class... Args>
diff --git a/dune/vtk/pvdwriter.impl.hh b/dune/vtk/pvdwriter.impl.hh
index 347eaeb..7f84bd1 100644
--- a/dune/vtk/pvdwriter.impl.hh
+++ b/dune/vtk/pvdwriter.impl.hh
@@ -10,7 +10,7 @@ namespace Dune {
 
 template <class W>
 void PvdWriter<W>
-  ::writeTimestep (double time, std::string const& fn, Std::optional<std::string> dir, bool writeCollection) const
+  ::writeTimestep (double time, std::string const& fn, std::optional<std::string> dir, bool writeCollection) const
 {
   auto p = filesystem::path(fn);
   auto name = p.stem();
@@ -50,7 +50,7 @@ void PvdWriter<W>
 
 template <class W>
 std::string PvdWriter<W>
-  ::write (std::string const& fn, Std::optional<std::string> /*dir*/) const
+  ::write (std::string const& fn, std::optional<std::string> /*dir*/) const
 {
   auto p = filesystem::path(fn);
   auto name = p.stem();
diff --git a/dune/vtk/vtkfunction.hh b/dune/vtk/vtkfunction.hh
index 48444ec..d51f64c 100644
--- a/dune/vtk/vtkfunction.hh
+++ b/dune/vtk/vtkfunction.hh
@@ -1,8 +1,8 @@
 #pragma once
 
+#include <optional>
 #include <type_traits>
 
-#include <dune/common/std/optional.hh>
 #include <dune/common/std/type_traits.hh>
 
 #include "vtklocalfunction.hh"
@@ -52,7 +52,7 @@ namespace Dune
      * \param type  The VTK datatype how to write the function values to the output [Vtk::FLOAT64]
      **/
     VtkFunction (std::shared_ptr<VTKFunction<GridView> const> const& fct,
-                 Std::optional<Vtk::DataTypes> type = {})
+                 std::optional<Vtk::DataTypes> type = {})
       : localFct_(fct)
       , name_(fct->name())
       , ncomps_(fct->ncomps())
@@ -73,8 +73,8 @@ namespace Dune
     template <class F,
       class = void_t<LocalFunction<F>> >
     VtkFunction (F&& fct, std::string name,
-                 Std::optional<int> ncomps = {},
-                 Std::optional<Vtk::DataTypes> type = {})
+                 std::optional<int> ncomps = {},
+                 std::optional<Vtk::DataTypes> type = {})
       : localFct_(localFunction(std::forward<F>(fct)))
       , name_(std::move(name))
     {
@@ -88,7 +88,7 @@ namespace Dune
     template <class F,
       class = void_t<LocalFunction<F>> >
     VtkFunction (F&& fct, Vtk::FieldInfo fieldInfo,
-                 Std::optional<Vtk::DataTypes> type = {})
+                 std::optional<Vtk::DataTypes> type = {})
       : VtkFunction(std::forward<F>(fct), fieldInfo.name(), fieldInfo.ncomps(), type)
     {}
 
diff --git a/dune/vtk/vtktimeserieswriter.hh b/dune/vtk/vtktimeserieswriter.hh
index 21bcd6c..bbac2b3 100644
--- a/dune/vtk/vtktimeserieswriter.hh
+++ b/dune/vtk/vtktimeserieswriter.hh
@@ -1,12 +1,11 @@
 #pragma once
 
 #include <cassert>
+#include <optional>
 #include <string>
 #include <tuple>
 #include <vector>
 
-#include <dune/common/std/optional.hh>
-
 #include <dune/vtk/filewriter.hh>
 #include <dune/vtk/forward.hh>
 #include <dune/vtk/vtktypes.hh>
@@ -66,7 +65,7 @@ namespace Dune
      * \param writeCollection  Create a timeseries file directly
      **/
     void writeTimestep (double time, std::string const& fn,
-                        Std::optional<std::string> tmpDir = {},
+                        std::optional<std::string> tmpDir = {},
                         bool writeCollection = true) const;
 
     /// Writes all timesteps to single timeseries file.
@@ -79,7 +78,7 @@ namespace Dune
      * 
      * \returns File name of the actual written file
      **/
-    virtual std::string write (std::string const& fn, Std::optional<std::string> dir = {}) const override;
+    virtual std::string write (std::string const& fn, std::optional<std::string> dir = {}) const override;
 
     /// Attach point data to the writer, \see VtkFunction for possible arguments
     template <class Function, class... Args>
diff --git a/dune/vtk/vtktimeserieswriter.impl.hh b/dune/vtk/vtktimeserieswriter.impl.hh
index 96cacb9..f3822da 100644
--- a/dune/vtk/vtktimeserieswriter.impl.hh
+++ b/dune/vtk/vtktimeserieswriter.impl.hh
@@ -39,7 +39,7 @@ VtkTimeseriesWriter<W>::~VtkTimeseriesWriter ()
 
 template <class W>
 void VtkTimeseriesWriter<W>
-  ::writeTimestep (double time, std::string const& fn, Std::optional<std::string> tmpDir, bool writeCollection) const
+  ::writeTimestep (double time, std::string const& fn, std::optional<std::string> tmpDir, bool writeCollection) const
 {
   auto name = filesystem::path(fn).stem();
   auto tmpBase = tmpDir ? filesystem::path(*tmpDir) : tmpDir_;
@@ -75,7 +75,7 @@ void VtkTimeseriesWriter<W>
 
 template <class W>
 std::string VtkTimeseriesWriter<W>
-  ::write (std::string const& fn, Std::optional<std::string> dir) const
+  ::write (std::string const& fn, std::optional<std::string> dir) const
 {
   assert( initialized_ );
 
diff --git a/dune/vtk/vtkwriterinterface.hh b/dune/vtk/vtkwriterinterface.hh
index b379a0f..8dadb0a 100644
--- a/dune/vtk/vtkwriterinterface.hh
+++ b/dune/vtk/vtkwriterinterface.hh
@@ -3,11 +3,11 @@
 #include <iosfwd>
 #include <map>
 #include <memory>
+#include <optional>
 #include <string>
 #include <vector>
 
 #include <dune/common/parallel/mpihelper.hh>
-#include <dune/common/std/optional.hh>
 #include <dune/vtk/filewriter.hh>
 #include <dune/vtk/forward.hh>
 #include <dune/vtk/vtkfunction.hh>
@@ -84,7 +84,7 @@ namespace Dune
      * 
      * \returns File name that is actually written.
      **/
-    virtual std::string write (std::string const& fn, Std::optional<std::string> dir = {}) const override;
+    virtual std::string write (std::string const& fn, std::optional<std::string> dir = {}) const override;
 
     /// \brief Attach point data to the writer
     /**
@@ -158,7 +158,7 @@ namespace Dune
                     std::vector<pos_type>& offsets,
                     VtkFunction const& fct,
                     PositionTypes type,
-                    Std::optional<std::size_t> timestep = {}) const;
+                    std::optional<std::size_t> timestep = {}) const;
 
     // Write point-data and cell-data in raw/compressed format to output stream
     void writeDataAppended (std::ofstream& out, std::vector<std::uint64_t>& blocks) const;
@@ -168,7 +168,7 @@ namespace Dune
     // vector `offsets`.
     void writePoints (std::ofstream& out,
                       std::vector<pos_type>& offsets,
-                      Std::optional<std::size_t> timestep = {}) const;
+                      std::optional<std::size_t> timestep = {}) const;
 
     // Write Appended section and fillin offset values to XML attributes
     void writeAppended (std::ofstream& out, std::vector<pos_type> const& offsets) const;
diff --git a/dune/vtk/vtkwriterinterface.impl.hh b/dune/vtk/vtkwriterinterface.impl.hh
index f0e7715..597774c 100644
--- a/dune/vtk/vtkwriterinterface.impl.hh
+++ b/dune/vtk/vtkwriterinterface.impl.hh
@@ -24,7 +24,7 @@ namespace Dune {
 
 template <class GV, class DC>
 std::string VtkWriterInterface<GV,DC>
-  ::write (std::string const& fn, Std::optional<std::string> dir) const
+  ::write (std::string const& fn, std::optional<std::string> dir) const
 {
   dataCollector_->update();
 
@@ -80,7 +80,7 @@ template <class GV, class DC>
 void VtkWriterInterface<GV,DC>
   ::writeData (std::ofstream& out, std::vector<pos_type>& offsets,
                VtkFunction const& fct, PositionTypes type,
-               Std::optional<std::size_t> timestep) const
+               std::optional<std::size_t> timestep) const
 {
   out << "<DataArray Name=\"" << fct.name() << "\" type=\"" << to_string(fct.type()) << "\""
       << " NumberOfComponents=\"" << fct.ncomps() << "\" format=\"" << (format_ == Vtk::ASCII ? "ascii\"" : "appended\"");
@@ -106,7 +106,7 @@ void VtkWriterInterface<GV,DC>
 template <class GV, class DC>
 void VtkWriterInterface<GV,DC>
   ::writePoints (std::ofstream& out, std::vector<pos_type>& offsets,
-                Std::optional<std::size_t> timestep) const
+                std::optional<std::size_t> timestep) const
 {
   out << "<DataArray type=\"" << to_string(datatype_) << "\""
       << " NumberOfComponents=\"3\" format=\"" << (format_ == Vtk::ASCII ? "ascii\"" : "appended\"");
diff --git a/dune/vtk/writers/vtkrectilineargridwriter.hh b/dune/vtk/writers/vtkrectilineargridwriter.hh
index 32e2e4b..f119a05 100644
--- a/dune/vtk/writers/vtkrectilineargridwriter.hh
+++ b/dune/vtk/writers/vtkrectilineargridwriter.hh
@@ -42,7 +42,7 @@ namespace Dune
     virtual void writeParallelFile (std::ofstream& out, std::string const& pfilename, int size) const override;
 
     void writeCoordinates (std::ofstream& out, std::vector<pos_type>& offsets,
-                           Std::optional<std::size_t> timestep = {}) const;
+                           std::optional<std::size_t> timestep = {}) const;
 
     template <class T>
     std::array<std::uint64_t, 3> writeCoordinatesAppended (std::ofstream& out) const;
diff --git a/dune/vtk/writers/vtkrectilineargridwriter.impl.hh b/dune/vtk/writers/vtkrectilineargridwriter.impl.hh
index d7be66f..f2c8223 100644
--- a/dune/vtk/writers/vtkrectilineargridwriter.impl.hh
+++ b/dune/vtk/writers/vtkrectilineargridwriter.impl.hh
@@ -116,7 +116,7 @@ void VtkRectilinearGridWriter<GV,DC>
 template <class GV, class DC>
 void VtkRectilinearGridWriter<GV,DC>
   ::writeCoordinates (std::ofstream& out, std::vector<pos_type>& offsets,
-                      Std::optional<std::size_t> timestep) const
+                      std::optional<std::size_t> timestep) const
 {
   std::string names = "xyz";
   if (format_ == Vtk::ASCII) {
diff --git a/dune/vtk/writers/vtkunstructuredgridwriter.hh b/dune/vtk/writers/vtkunstructuredgridwriter.hh
index e1732cb..a1d557b 100644
--- a/dune/vtk/writers/vtkunstructuredgridwriter.hh
+++ b/dune/vtk/writers/vtkunstructuredgridwriter.hh
@@ -74,11 +74,11 @@ namespace Dune
     // vector `offsets`.
     void writeCells (std::ofstream& out,
                      std::vector<pos_type>& offsets,
-                     Std::optional<std::size_t> timestep = {}) const;
+                     std::optional<std::size_t> timestep = {}) const;
 
     void writePointIds (std::ofstream& out,
                         std::vector<pos_type>& offsets,
-                        Std::optional<std::size_t> timestep = {}) const;
+                        std::optional<std::size_t> timestep = {}) const;
 
   private:
     using Super::dataCollector_;
diff --git a/dune/vtk/writers/vtkunstructuredgridwriter.impl.hh b/dune/vtk/writers/vtkunstructuredgridwriter.impl.hh
index 8819e22..7eb17d4 100644
--- a/dune/vtk/writers/vtkunstructuredgridwriter.impl.hh
+++ b/dune/vtk/writers/vtkunstructuredgridwriter.impl.hh
@@ -281,7 +281,7 @@ void VtkUnstructuredGridWriter<GV,DC>
 template <class GV, class DC>
 void VtkUnstructuredGridWriter<GV,DC>
   ::writeCells (std::ofstream& out, std::vector<pos_type>& offsets,
-                Std::optional<std::size_t> timestep) const
+                std::optional<std::size_t> timestep) const
 {
   if (format_ == Vtk::ASCII) {
     auto cells = dataCollector_->cells();
@@ -337,7 +337,7 @@ template <class GV, class DC>
 void VtkUnstructuredGridWriter<GV,DC>
   ::writePointIds (std::ofstream& out,
                    std::vector<pos_type>& offsets,
-                   Std::optional<std::size_t> timestep) const
+                   std::optional<std::size_t> timestep) const
 {
   auto ids = dataCollector_->pointIds();
   if (ids.empty())
-- 
GitLab