From 0754ccfbe46ae4cd7c19726029785d1d4baaae5d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20Omn=C3=A8s?= <florian.omnes@rte-france.com>
Date: Mon, 27 Sep 2021 17:52:46 +0200
Subject: [PATCH] Clean opt code a bit (#387)

* Remove unused OPT_ChargerLaContrainteAPartirDePi
* Use references instead of copies
---
 ...ruction_matrice_des_contraintes_outils.cpp | 53 ++-----------------
 src/solver/optimisation/opt_fonctions.h       |  1 -
 2 files changed, 3 insertions(+), 51 deletions(-)

diff --git a/src/solver/optimisation/opt_construction_matrice_des_contraintes_outils.cpp b/src/solver/optimisation/opt_construction_matrice_des_contraintes_outils.cpp
index 3ed7e0e1b3..9919ee7949 100644
--- a/src/solver/optimisation/opt_construction_matrice_des_contraintes_outils.cpp
+++ b/src/solver/optimisation/opt_construction_matrice_des_contraintes_outils.cpp
@@ -33,46 +33,6 @@
 
 #include "opt_fonctions.h"
 
-void OPT_ChargerLaContrainteAPartirDePi(PROBLEME_ANTARES_A_RESOUDRE* ProblemeAResoudre,
-                                        double* Pi,
-                                        char SensContrainte)
-{
-    int NombreDeVariables;
-    int NombreDeContraintes;
-    int NombreDeTermes;
-    int NombreDeTermesDeLaContrainte;
-    int Var;
-
-    NombreDeVariables = ProblemeAResoudre->NombreDeVariables;
-    NombreDeContraintes = ProblemeAResoudre->NombreDeContraintes;
-    NombreDeTermes = ProblemeAResoudre->NombreDeTermesDansLaMatriceDesContraintes;
-
-    NombreDeTermesDeLaContrainte = 0;
-    ProblemeAResoudre->IndicesDebutDeLigne[NombreDeContraintes] = NombreDeTermes;
-    for (Var = 0; Var < NombreDeVariables; Var++)
-    {
-        if (Pi[Var] == 0.0)
-            continue;
-        ProblemeAResoudre->CoefficientsDeLaMatriceDesContraintes[NombreDeTermes] = Pi[Var];
-        ProblemeAResoudre->IndicesColonnes[NombreDeTermes] = Var;
-        NombreDeTermesDeLaContrainte++;
-        NombreDeTermes++;
-        if (NombreDeTermes == ProblemeAResoudre->NombreDeTermesAllouesDansLaMatriceDesContraintes)
-        {
-            OPT_AugmenterLaTailleDeLaMatriceDesContraintes(ProblemeAResoudre);
-        }
-    }
-    ProblemeAResoudre->NombreDeTermesDesLignes[NombreDeContraintes] = NombreDeTermesDeLaContrainte;
-
-    ProblemeAResoudre->Sens[NombreDeContraintes] = SensContrainte;
-    NombreDeContraintes++;
-
-    ProblemeAResoudre->NombreDeContraintes = NombreDeContraintes;
-    ProblemeAResoudre->NombreDeTermesDansLaMatriceDesContraintes = NombreDeTermes;
-
-    return;
-}
-
 void OPT_ChargerLaContrainteDansLaMatriceDesContraintes(
   PROBLEME_ANTARES_A_RESOUDRE* ProblemeAResoudre,
   double* Pi,
@@ -80,15 +40,11 @@ void OPT_ChargerLaContrainteDansLaMatriceDesContraintes(
   int NombreDeTermesDeLaContrainte,
   char SensContrainte)
 {
-    int i;
-    int NombreDeTermes;
-    int NombreDeContraintes;
-
-    NombreDeContraintes = ProblemeAResoudre->NombreDeContraintes;
-    NombreDeTermes = ProblemeAResoudre->NombreDeTermesDansLaMatriceDesContraintes;
+    int& NombreDeTermes = ProblemeAResoudre->NombreDeTermesDansLaMatriceDesContraintes;
+    int& NombreDeContraintes = ProblemeAResoudre->NombreDeContraintes;
 
     ProblemeAResoudre->IndicesDebutDeLigne[NombreDeContraintes] = NombreDeTermes;
-    for (i = 0; i < NombreDeTermesDeLaContrainte; i++)
+    for (int i = 0; i < NombreDeTermesDeLaContrainte; i++)
     {
         ProblemeAResoudre->CoefficientsDeLaMatriceDesContraintes[NombreDeTermes] = Pi[i];
         ProblemeAResoudre->IndicesColonnes[NombreDeTermes] = Colonne[i];
@@ -103,8 +59,5 @@ void OPT_ChargerLaContrainteDansLaMatriceDesContraintes(
     ProblemeAResoudre->Sens[NombreDeContraintes] = SensContrainte;
     NombreDeContraintes++;
 
-    ProblemeAResoudre->NombreDeContraintes = NombreDeContraintes;
-    ProblemeAResoudre->NombreDeTermesDansLaMatriceDesContraintes = NombreDeTermes;
-
     return;
 }
diff --git a/src/solver/optimisation/opt_fonctions.h b/src/solver/optimisation/opt_fonctions.h
index b91ae33edb..b6b48a6866 100644
--- a/src/solver/optimisation/opt_fonctions.h
+++ b/src/solver/optimisation/opt_fonctions.h
@@ -73,7 +73,6 @@ void OPT_RestaurerLesDonnees(PROBLEME_HEBDO*);
 void OPT_CalculerLesPminThermiquesEnFonctionDeMUTetMDT(PROBLEME_HEBDO*);
 double OPT_CalculerAireMaxPminJour(int, int, int, int, int*, int*);
 
-void OPT_ChargerLaContrainteAPartirDePi(PROBLEME_ANTARES_A_RESOUDRE*, double*, char);
 void OPT_ChargerLaContrainteDansLaMatriceDesContraintes(PROBLEME_ANTARES_A_RESOUDRE*,
                                                         double*,
                                                         int*,