use a pair

AntaresSimulatorTeam · Dec 31, 2024 · fba0351 · fba0351
1 parent 8057164
commit fba0351
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/...libs/antares/study/include/antares/study/parts/short-term-storage/AdditionalConstraints.h b/...libs/antares/study/include/antares/study/parts/short-term-storage/AdditionalConstraints.h
@@ -47,13 +47,7 @@ struct AdditionalConstraints
 
     std::vector<SingleAdditionalConstraint> constraints;
 
-    struct ValidateResult
-    {
-        bool ok;
-        std::string error_msg;
-    };
-
-    ValidateResult validate() const;
+    std::pair<bool, std::string> validate() const;
 
 private:
     bool isValidVariable() const;

diff --git a/src/libs/antares/study/parts/short-term-storage/AdditionalConstraints.cpp b/src/libs/antares/study/parts/short-term-storage/AdditionalConstraints.cpp
@@ -18,13 +18,15 @@
 ** You should have received a copy of the Mozilla Public Licence 2.0
 ** along with Antares_Simulator. If not, see <https://opensource.org/license/mpl-2-0/>.
 */
-#include "antares/study/parts/short-term-storage/AdditionalConstraints.h"
 
 #include <algorithm>
 
+#include "antares/study/parts/short-term-storage/AdditionalConstraints.h"
+
 namespace Antares::Data::ShortTermStorage
 {
-AdditionalConstraints::ValidateResult AdditionalConstraints::validate() const
+
+std::pair<bool, std::string> AdditionalConstraints::validate() const
 {
     if (cluster_id.empty())
     {