Fix compiler warnings in Parallel library

Mainly:
- add variable initializasation in class constructors
- cast int operands in arithmetic operations when the result is a longint
- plus some minor formating errors detected during pre-commit

src/Parallel/PLMPI/PLMPIMaster.cpp

@@ -7,15 +7,25 @@
 PLMPIMaster::PLMPIMaster(PLParallelTask* t)
 {
 	ALString sTmp;
-	bInterruptionRequested = false;
-	bIsMaxErrorReached = 0;
-	bIsMaxWarningReached = 0;
-	bIsMaxMessageReached = 0;
 	nWorkingSlaves = 0;
 	task = t;
+	bNewMessage = false;
 	bSpawnedDone = false;
+	bStopOrderDone = false;
+	bInterruptionRequested = false;
+	bSlaveError = false;
+	bMasterError = false;
+	bIsMaxErrorReached = false;
+	bIsMaxWarningReached = false;
+	bIsMaxMessageReached = false;
+	dGlobalProgression = 0;
+	nOldProgression = 0;
+	bIsProcessing = false;
+	nInitialisationCount = 0;
+	nFinalisationCount = 0;
 	nFirstSlaveInitializeMessageRank = -1;
 	nFirstSlaveFinalizeMessageRank = -1;
+	sBufferDischarge[0] = '\0';
 }
 
 PLMPIMaster::~PLMPIMaster()
@@ -223,7 +233,7 @@ boolean PLMPIMaster::Run()
 		slave->SetRank(context.GetRank());
 		slave->SetHostName(sHostName);
 		slave->SetProgression(0);
-		slave->SetTaskPercent(1 / task->nWorkingProcessNumber); // Pour l'initialisation
+		slave->SetTaskPercent(1.0 / task->nWorkingProcessNumber); // Pour l'initialisation
 		GetTask()->oaSlaves.Add(slave);
 
 		// Ajout de l'esclave dans le dictionaire hsostName / liste des esclaves

src/Parallel/PLMPI/PLMPIMessageManager.cpp

@@ -17,12 +17,12 @@ int MessageWithTaskIndexCompare(const void* elem1, const void* elem2)
 	message2 = cast(MessageWithTaskIndex*, *(Object**)elem2);
 
 	// Comparaison sur le TaskIndex puis sur le numero de ligne, puis sur l'ordre de reception
-	lIndexComp = message1->GetTaskIndex() - message2->GetTaskIndex();
+	lIndexComp = (longint)message1->GetTaskIndex() - message2->GetTaskIndex();
 	if (lIndexComp == 0)
 	{
 		lIndexComp = message1->GetMessage()->GetIndex() - message2->GetMessage()->GetIndex();
 		if (lIndexComp == 0)
-			lIndexComp = message1->GetAddIndex() - message2->GetAddIndex();
+			lIndexComp = (longint)message1->GetAddIndex() - message2->GetAddIndex();
 	}
 	return (int)((lIndexComp > 0) - (lIndexComp < 0)); // 1 si > 0; -1 si >0; 0 sinon
 }

src/Parallel/PLMPI/PLMPISlaveProgressionManager.cpp

@@ -16,6 +16,8 @@ PLMPISlaveProgressionManager::PLMPISlaveProgressionManager(void)
 	nOldProgression = 0;
 	sendRequest = MPI_REQUEST_NULL;
 	ivMaxErrorReached = NULL;
+	nSlaveState = State::VOID;
+	sBuffer[0] = '\0';
 }
 
 PLMPISlaveProgressionManager::~PLMPISlaveProgressionManager(void)

src/Parallel/PLParallelTask/PLFileConcatenater.cpp

@@ -170,7 +170,7 @@ boolean PLFileConcatenater::Concatenate(const StringVector* svChunkURIs, const O
 		nInputPreferredSize = SystemFile::nMinPreferredBufferSize;
 	nOutputPreferredSize = PLRemoteFileService::GetPreferredBufferSize(sOutputFileName);
 
-	if (lRemainingMemory >= nInputPreferredSize + nOutputPreferredSize)
+	if (lRemainingMemory >= (longint)nInputPreferredSize + nOutputPreferredSize)
 	{
 		nOutputBufferSize = nOutputPreferredSize;
 		lInputBufferSize = lRemainingMemory - nOutputBufferSize;
@@ -180,8 +180,8 @@ boolean PLFileConcatenater::Concatenate(const StringVector* svChunkURIs, const O
 			lInputBufferSize = (lInputBufferSize / nInputPreferredSize) * nInputPreferredSize;
 
 		// On n'utilise pas plus de 8 preferred size
-		if (lInputBufferSize > 8 * nOutputPreferredSize)
-			lInputBufferSize = 8 * nOutputPreferredSize;
+		if (lInputBufferSize > 8 * (longint)nOutputPreferredSize)
+			lInputBufferSize = 8 * (longint)nOutputPreferredSize;
 	}
 	else
 	{

src/Parallel/PLParallelTask/PLParallelTask.cpp

@@ -2129,7 +2129,7 @@ int PLParallelTask::InternalComputeStairBufferSize(int nBufferSizeMin, int nBuff
 
 	// Fin de traitement : quand il n'y a du travail que pour la moitie des esclaves
 	lFileSizeRemaining = lFileSize - lFileProcessed;
-	if (lFileSizeRemaining < (nProcessNumber / 2) * nComputedBufferSizeMax)
+	if (lFileSizeRemaining < (nProcessNumber / 2) * (longint)nComputedBufferSizeMax)
 	{
 		bStop = true;
 	}

src/Parallel/PLParallelTask/PLSharedVector.cpp

@@ -233,10 +233,10 @@ void PLShared_LongintVector::Test()
 
 	// Initialisation d'une variable partagee en entree
 	for (i = 0; i < 5; i++)
-		ivInitialValue.Add(i + 1);
+		ivInitialValue.Add((longint)i + 1);
 	shared_valueIn.GetLongintVector()->CopyFrom(&ivInitialValue);
 	for (i = 0; i < 5; i++)
-		shared_valueIn.Add(101 + i);
+		shared_valueIn.Add(101 + (longint)i);
 
 	// Serialisation
 	serializer.OpenForWrite(NULL);

src/Parallel/PLParallelTask/RMParallelResourceManager.cpp

@@ -2357,7 +2357,7 @@ boolean PLSolution::FitMinimalRequirements(int nRT, longint& lMissingResource, b
 					if (hostClassSolution->GetHostCountPerProcNumber()->GetAt(nProcNumber) != 0)
 					{
 						// Resources necessaires sur ce host
-						lUsedResource = lMasterSum + (nProcNumber - 1) * lSlaveMin;
+						lUsedResource = lMasterSum + ((longint)nProcNumber - 1) * lSlaveMin;
 
 						// Ressources disponibles sur le host
 						lLocalMissingResource = lUsedResource - lAvailableResource;

src/Parallel/PLSamples/PEFileSearchTask.cpp

@@ -20,6 +20,7 @@ PEFileSearchTask::PEFileSearchTask()
 	// Variables du maitre
 	lInputFileSize = 0;
 	lFoundLineNumber = 0;
+	lFilePos = 0;
 }
 
 PEFileSearchTask::~PEFileSearchTask() {}

src/Parallel/PLSamples/PEIOParallelTestTask.cpp

@@ -15,6 +15,7 @@ PEIOParallelTestTask::PEIOParallelTestTask()
 	lTotalFieldNumber = 0;
 	lTotalLineNumber = 0;
 	lTotalFileSizes = 0;
+	nFileIndex = 0;
 }
 
 PEIOParallelTestTask::~PEIOParallelTestTask() {}

src/Parallel/PLSamples/PELullaby.cpp

@@ -4,7 +4,11 @@
 
 #include "PELullabyTask.h"
 
-PELullabyTask::PELullabyTask() {}
+PELullabyTask::PELullabyTask()
+{
+	nMyTaskIndex = 0;
+	nStepSize = 0;
+}
 
 PELullabyTask::~PELullabyTask() {}
 

src/Parallel/PLSamples/PEProgressionTask.cpp

@@ -8,6 +8,7 @@ PEProgressionTask::PEProgressionTask()
 {
 	nMethodToTest = 0;
 	bBoostMode = false;
+	nIterationNumber = 0;
 	DeclareTaskInput(&input_nIterationNumber);
 }
 PEProgressionTask::~PEProgressionTask() {}
@@ -74,7 +75,7 @@ boolean PEProgressionTask::Test(int nMethod, boolean bValue)
 boolean PEProgressionTask::Test()
 {
 	int nMethod;
-	int bOk;
+	boolean bOk;
 	bOk = true;
 	for (nMethod = MASTER_INITIALIZE; nMethod < END; nMethod++)
 	{

src/Parallel/PLSamples/PESerializerLongTestTask.cpp

@@ -8,6 +8,7 @@ PESerializerLongTestTask::PESerializerLongTestTask()
 {
 	DeclareTaskInput(&input_sString);
 	DeclareTaskInput(&input_nStringLength);
+	nMaxStringSize = 0;
 }
 
 PESerializerLongTestTask::~PESerializerLongTestTask() {}

src/Parallel/PLSamples/PESerializerTestTask.cpp

@@ -18,6 +18,8 @@ PESerializerTestTask::PESerializerTestTask()
 	DeclareSharedParameter(&shared_nLargeCharVectorSize);
 	DeclareSharedParameter(&shared_nLargeStringSize);
 
+	nLargeCharVectorSize = 0;
+	nLargeStringSize = 0;
 	sSimpleStringToSerialize = "A simple string";
 	sSimpleCharVectorToSerialize = "A simple CharVector !!";
 }

test/LearningTestTool/py/kht_export.py

@@ -146,9 +146,7 @@ def export_learning_test_tree(
                         # Creation du repertoire de suite uniquement si necessaire
                         if not target_suite_dir_created:
                             utils.make_dir(target_suite_dir)
-                            target_suite_dir_created = os.path.isdir(
-                                target_suite_dir
-                            )
+                            target_suite_dir_created = os.path.isdir(target_suite_dir)
                         # Cas d'un repertoire
                         if os.path.isdir(source_test_dir):
                             # Copie du repertoire de test