Merge pull request #3310 from akva2/use_fmt

EModel: use {fmt}

src/opm/common/OpmLog/CounterLog.cpp

@@ -16,15 +16,12 @@
   You should have received a copy of the GNU General Public License
   along with OPM.  If not, see <http://www.gnu.org/licenses/>.
  */
-#include <stdexcept>
-#include <sstream>
-#include <cassert>
 
 #include <opm/common/OpmLog/OpmLog.hpp>
 #include <opm/common/OpmLog/LogUtil.hpp>
 #include <opm/common/OpmLog/CounterLog.hpp>
 
-
+#include <stdexcept>
 
 namespace Opm {
 

src/opm/input/eclipse/Parser/Parser.cpp

@@ -19,9 +19,7 @@
 
 #include <cctype>
 #include <filesystem>
-#include <fstream>
 #include <iterator>
-#include <iomanip>
 #include <iostream>
 #include <optional>
 #include <stack>

src/opm/input/eclipse/Schedule/Action/PyAction.cpp

@@ -16,7 +16,6 @@
   You should have received a copy of the GNU General Public License
   along with OPM.  If not, see <http://www.gnu.org/licenses/>.
 */
-#include <fstream>
 #include <memory>
 
 #ifdef EMBEDDED_PYTHON

src/opm/input/eclipse/Schedule/ArrayDimChecker.cpp

@@ -19,7 +19,6 @@
 #include <algorithm>
 #include <cstddef>
 #include <iterator>
-#include <sstream>
 
 #include <fmt/format.h>
 

src/opm/io/eclipse/EGrid.cpp

@@ -23,6 +23,7 @@
 #include <opm/common/ErrorMacros.hpp>
 
 #include <algorithm>
+#include <cmath>
 #include <cstring>
 #include <filesystem>
 #include <fstream>
@@ -30,11 +31,8 @@
 #include <iomanip>
 #include <numeric>
 #include <string>
-#include <sstream>
 #include <stdexcept>
 
-#include <math.h>
-
 namespace Opm { namespace EclIO {
 
 using NNCentry = std::tuple<int, int, int, int, int,int, float>;

src/opm/io/eclipse/ERft.cpp

@@ -22,10 +22,8 @@
 
 #include <algorithm>
 #include <cstring>
-#include <iomanip>
 #include <iterator>
 #include <string>
-#include <sstream>
 
 namespace Opm { namespace EclIO {
 

src/opm/io/eclipse/ERst.cpp

@@ -23,10 +23,8 @@
 #include <algorithm>
 #include <cstring>
 #include <exception>
-#include <iomanip>
 #include <iterator>
 #include <regex>
-#include <sstream>
 #include <stdexcept>
 #include <string>
 

src/opm/io/eclipse/EclFile.cpp

@@ -28,7 +28,6 @@
 #include <fstream>
 #include <iomanip>
 #include <iterator>
-#include <sstream>
 #include <string>
 #include <numeric>
 #include <cmath>

src/opm/output/eclipse/AggregateUDQData.cpp

@@ -42,7 +42,6 @@
 #include <cstring>
 #include <fmt/format.h>
 #include <iostream>
-#include <sstream>
 #include <string>
 
 

src/opm/output/eclipse/RestartIO.cpp

@@ -56,11 +56,8 @@
 #include <algorithm>
 #include <cassert>
 #include <cstddef>
-#include <fstream>
 #include <initializer_list>
-#include <iomanip>
 #include <iterator>
-#include <sstream>
 #include <stdexcept>
 #include <string>
 #include <unordered_set>

src/opm/output/eclipse/Summary.cpp

@@ -77,7 +77,6 @@
 #include <memory>
 #include <numeric>
 #include <regex>
-#include <sstream>
 #include <stdexcept>
 #include <string>
 #include <unordered_map>

src/opm/utility/EModel.cpp

@@ -21,14 +21,11 @@
 
 #include <fmt/format.h>
 
-#include <filesystem>
-#include <string>
-#include <string.h>
-#include <sstream>
-#include <iterator>
-#include <iomanip>
 #include <algorithm>
-
+#include <cstring>
+#include <filesystem>
+#include <iterator>
+#include <string>
 
 using EclEntry = std::tuple<std::string, Opm::EclIO::eclArrType, long int>;
 using ParamEntry = std::tuple<std::string, Opm::EclIO::eclArrType>;
@@ -191,8 +188,10 @@ void EModel::initSolutionData(int rstep){
 void EModel::get_cell_volumes_from_grid()
 {
     if (!grid.has_value()){
-        std::string message = "\nnot possible to calculate cell columes without an Egrid file. ";
-        message = message + "The grid file must have same root name as the init file selected for this object";
+        std::string message = "\nNot possible to calculate cell volumes "
+                              "without an Egrid file. "
+                              "The grid file must have same root name as "
+                              "the init file selected for this object";
         throw std::runtime_error(message);
     }
 
@@ -279,7 +278,8 @@ void EModel::updateActiveFilter(const std::vector<T>& paramVect, const std::stri
                 ActFilter[i] = false;
 
     } else {
-        std::string message = "Unknown opprator " + opperator + ", used to set filter";
+        const std::string message =
+            fmt::format("Unknown operator {} used to set filter", opperator);
         throw std::invalid_argument(message);
     }
 
@@ -295,7 +295,9 @@ void EModel::updateActiveFilter(const std::vector<T>& paramVect, const std::stri
                 ActFilter[i] = false;
 
     } else {
-        throw std::invalid_argument("Unknown opprator " + opperator + ", used to set filter");
+        const std::string message =
+            fmt::format("Unknown operator {} used to set filter", opperator);
+        throw std::invalid_argument(message);
     }
 
     activeFilter = true;
@@ -314,7 +316,10 @@ const std::vector<T>& EModel::get_filter_param(const std::string& param)
         else if (hasInitParameter(param))
             return initfile.get<int>(param);
 
-        throw std::invalid_argument("parameter " + param + ", used to set filter, could not be found");
+        const std::string message =
+            fmt::format("parameter {}, used to set filter, could not be found",
+                        param);
+        throw std::invalid_argument(message);
 
     } else if constexpr (std::is_same<T, float>::value){
         if (param == "PORV")
@@ -329,7 +334,10 @@ const std::vector<T>& EModel::get_filter_param(const std::string& param)
         else if (hasSolutionParameter(param))
             return getSolutionFloat(param);
 
-        throw std::invalid_argument("parameter " + param + ", used to set filter, could not be found");
+        const std::string message =
+            fmt::format("parameter {}, used to set filter, could not be found",
+                        param);
+        throw std::invalid_argument(message);
     }
 }
 
@@ -366,8 +374,10 @@ void EModel::addFilter<float>(const std::string& param1, const std::string& oppe
 
 void EModel::addHCvolFilter()
 {
-    if (FreeWaterlevel.size()==0)
-        throw std::runtime_error("free water level needs to be inputted via function setDepthfwl before using filter HC filter");
+    if (FreeWaterlevel.empty())
+        throw std::runtime_error("free water level needs to be input via "
+                                 "function setDepthfwl before using "
+                                 "filter HC filter");
 
     auto eqlnum = initfile.get<int>("EQLNUM");
     auto depth = initfile.get<float>("DEPTH");
@@ -435,7 +445,9 @@ const std::vector<float>& EModel::getInitFloat(const std::string& name)
 const std::vector<float>& EModel::getSolutionFloat(const std::string& name)
 {
     if (!hasSolutionParameter(name)) {
-        std::string message = "parameter " + name + " not found for step 0 in restart file ";
+        const std::string message =
+            fmt::format("parameter {} not found for step 0 in restart file ",
+                        name);
         throw std::invalid_argument(message);
     }
 
@@ -456,7 +468,11 @@ void EModel::setDepthfwl(const std::vector<float>& fwl)
     int maxEqlnum = *it;
 
     if (maxEqlnum > nEqlnum){
-        std::string message= "FWL not defined for all eql regions. # Contacts input: " + std::to_string(nEqlnum) + " needed (max value in EQLNUM): " + std::to_string(maxEqlnum);
+        const std::string message =
+            fmt::format("FWL not defined for all eql regions. "
+                        "# Contacts input: {} needed "
+                        "(max value in EQLNUM): {}",
+                        nEqlnum, maxEqlnum);
         throw std::invalid_argument(message);
     }
 }