From: abn <adrien.bruneton@cea.fr>
Date: Tue, 22 Sep 2020 07:54:19 +0000 (+0200)
Subject: clang -Wconditional-uninitialized. Some remaining in SWIG generated code.
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=d9f2965d2ec41a024cf8a6882f0a483d403ce19e;p=tools%2Fmedcoupling.git

clang -Wconditional-uninitialized. Some remaining in SWIG generated code.
---

diff --git a/src/INTERP_KERNEL/DirectedBoundingBox.cxx b/src/INTERP_KERNEL/DirectedBoundingBox.cxx
index a70db19a4..2f03e8ad4 100644
--- a/src/INTERP_KERNEL/DirectedBoundingBox.cxx
+++ b/src/INTERP_KERNEL/DirectedBoundingBox.cxx
@@ -28,7 +28,6 @@
 #define __AXIS(i)     (&_axes[(i)*_dim])
 #define __MIN(i)      _minmax[i*2]
 #define __MAX(i)      _minmax[i*2+1]
-#define __MYID        (long(this)%10000)
 #define __DMP(msg) \
   //  cout << msg << endl
 
diff --git a/src/INTERP_KERNEL/Geometric2D/InterpKernelGeo2DQuadraticPolygon.cxx b/src/INTERP_KERNEL/Geometric2D/InterpKernelGeo2DQuadraticPolygon.cxx
index 9df9026a1..b6f888faf 100644
--- a/src/INTERP_KERNEL/Geometric2D/InterpKernelGeo2DQuadraticPolygon.cxx
+++ b/src/INTERP_KERNEL/Geometric2D/InterpKernelGeo2DQuadraticPolygon.cxx
@@ -540,10 +540,10 @@ void QuadraticPolygon::buildFromCrudeDataArray2(const std::map<mcIdType,INTERP_K
             {
               mcIdType idBg=direct?subEdge[2*j]:subEdge[2*nbOfSubEdges-2*j-1];
               mcIdType idEnd=direct?subEdge[2*j+1]:subEdge[2*nbOfSubEdges-2*j-2];
-              bool direction11,found=false;
-              bool direct1;//store if needed the direction in 1
-              mcIdType offset2;
-              mcIdType nbOfSubEdges1;
+              bool direction11=false,found=false;
+              bool direct1=false;//store if needed the direction in 1
+              mcIdType offset2=0;
+              mcIdType nbOfSubEdges1=0;
               for(std::vector<std::pair<mcIdType,std::pair<bool,mcIdType> > >::const_iterator it=idIns1.begin();it!=idIns1.end() && !found;it++)
                 {
                   mcIdType idIn1=(*it).first;//store if needed the cell id in 1
@@ -1089,7 +1089,7 @@ void QuadraticPolygon::ClosePolygons(std::list<QuadraticPolygon *>& pol1Zip, con
                                      std::vector<QuadraticPolygon *>& results)
 {
   bool directionKnownInPol2=false;
-  bool directionInPol2;
+  bool directionInPol2=false;
   bool needCleaning = false;
   for(std::list<QuadraticPolygon *>::iterator iter=pol1Zip.begin();iter!=pol1Zip.end();)
     {
diff --git a/src/INTERP_KERNEL/Interpolation3D.hxx b/src/INTERP_KERNEL/Interpolation3D.hxx
index 43290d9f5..9592252ba 100755
--- a/src/INTERP_KERNEL/Interpolation3D.hxx
+++ b/src/INTERP_KERNEL/Interpolation3D.hxx
@@ -40,8 +40,6 @@ namespace INTERP_KERNEL
     Interpolation3D(const InterpolationOptions& io);
     template<class MyMeshType, class MatrixType>
     typename MyMeshType::MyConnType interpolateMeshes(const MyMeshType& srcMesh, const MyMeshType& targetMesh, MatrixType& result, const std::string& method);
-  private:
-    SplittingPolicy _splitting_policy;
   };
 }
 
diff --git a/src/INTERP_KERNEL/PolygonAlgorithms.txx b/src/INTERP_KERNEL/PolygonAlgorithms.txx
index 1d4adc6c8..17f41408a 100644
--- a/src/INTERP_KERNEL/PolygonAlgorithms.txx
+++ b/src/INTERP_KERNEL/PolygonAlgorithms.txx
@@ -394,7 +394,7 @@ namespace INTERP_KERNEL
                                                                        int N1, int N2)
   {    
     int i_loc, i_glob, j1, j1_glob, j2,j2_glob, j3, j3_glob, j4,j4_glob,
-      i_prev, i_prev_glob, i_next, i_next_glob, nb_prev, sign, idim;
+      i_prev, i_prev_glob, i_next, i_next_glob, nb_prev=0, sign, idim;
     const double * Poly1, * Poly2;
     bool four_neighbours=false;
     _terminus = N1 < 3 || N2<3;
diff --git a/src/MEDCoupling/MEDCouplingCartesianAMRMesh.cxx b/src/MEDCoupling/MEDCouplingCartesianAMRMesh.cxx
index 808c41083..719fd48d1 100755
--- a/src/MEDCoupling/MEDCouplingCartesianAMRMesh.cxx
+++ b/src/MEDCoupling/MEDCouplingCartesianAMRMesh.cxx
@@ -897,7 +897,7 @@ bool FindInflection(const INTERP_KERNEL::BoxSplittingOptions& bso, const Interna
 {
   bool cutFound(false); cutPlace=-1;// do not set axisId before to be sure that cutFound was set to true
   const std::vector< std::pair<mcIdType,mcIdType> >& part(patchToBeSplit->getConstPart());
-  mcIdType sign,minimumPatchLength(bso.getMinimumPatchLength());
+  mcIdType sign=0,minimumPatchLength(bso.getMinimumPatchLength());
   const mcIdType dim(patchToBeSplit->getDimension());
 
   std::vector<mcIdType> zeroCrossDims(dim,-1);
diff --git a/src/MEDCoupling/MEDCouplingUMesh.cxx b/src/MEDCoupling/MEDCouplingUMesh.cxx
index 839051cf1..c022388b4 100755
--- a/src/MEDCoupling/MEDCouplingUMesh.cxx
+++ b/src/MEDCoupling/MEDCouplingUMesh.cxx
@@ -7362,7 +7362,7 @@ void MEDCouplingUMesh::TryToCorrectPolyhedronOrientation(mcIdType *begin, mcIdTy
           nbOfEdgesInFace=std::distance(bgFace,endFace);
           if(!isPerm[i])
             {
-              bool b;
+              bool b=false;
               for(std::size_t j=0;j<nbOfEdgesInFace;j++)
                 {
                   std::pair<mcIdType,mcIdType> p1(bgFace[j],bgFace[(j+1)%nbOfEdgesInFace]);
diff --git a/src/MEDLoader/MEDFileMeshLL.cxx b/src/MEDLoader/MEDFileMeshLL.cxx
index e9d3f2d14..72f4486e5 100644
--- a/src/MEDLoader/MEDFileMeshLL.cxx
+++ b/src/MEDLoader/MEDFileMeshLL.cxx
@@ -104,7 +104,7 @@ double MeshCls::checkMeshTimeStep(med_idt fid, const std::string& mName, int nst
 {
   bool found=false;
   med_int numdt,numit;
-  med_float dtt;
+  med_float dtt=0.0;
   std::vector< std::pair<int,int> > p(nstep);
   for(int i=0;i<nstep;i++)
     {
@@ -171,7 +171,7 @@ std::vector<const BigMemoryObject *> MEDFileMeshL2::getDirectChildrenWithNull()
 
 INTERP_KERNEL::AutoCppPtr<MeshOrStructMeshCls> MEDFileMeshL2::GetMeshIdFromName(med_idt fid, const std::string& mName, MEDCoupling::MEDCouplingMeshType& meshType, MEDCoupling::MEDCouplingAxisType& axType, int& dt, int& it, std::string& dtunit1)
 {
-  med_mesh_type type_maillage;
+  med_mesh_type type_maillage=MED_UNDEF_MESH_TYPE;
   char maillage_description[MED_COMMENT_SIZE+1];
   char dtunit[MED_LNAME_SIZE+1];
   med_int spaceDim,dim;
@@ -182,7 +182,7 @@ INTERP_KERNEL::AutoCppPtr<MeshOrStructMeshCls> MEDFileMeshL2::GetMeshIdFromName(
   med_sorting_type stype;
   std::vector<std::string> ms;
   med_int nstep;
-  med_axis_type axistype;
+  med_axis_type axistype=MED_UNDEF_AXIS_TYPE;
   for(int i=0;i<n && found==0;i++)
     {
       med_int naxis(MEDmeshnAxis(fid,i+1));
diff --git a/src/ParaMEDMEM/MPIAccessDEC.cxx b/src/ParaMEDMEM/MPIAccessDEC.cxx
index 21c128363..7ffa94195 100644
--- a/src/ParaMEDMEM/MPIAccessDEC.cxx
+++ b/src/ParaMEDMEM/MPIAccessDEC.cxx
@@ -335,7 +335,7 @@ namespace MEDCoupling
       {
         return allToAllTime( sendbuf, sendcount, sendtype , recvbuf, recvcount, recvtype ) ;
       }
-    int sts ;
+    int sts = 0;
     int target ;
     int sendoffset = 0 ;
     int recvoffset = 0 ;
@@ -397,7 +397,7 @@ namespace MEDCoupling
         return allToAllvTime( sendbuf, sendcounts, sdispls, sendtype ,
                               recvbuf, recvcounts, rdispls, recvtype ) ;
       }
-    int sts ;
+    int sts = 0;
     int target ;
     int SendRequestId ;
     int RecvRequestId ;
@@ -512,7 +512,7 @@ namespace MEDCoupling
   int MPIAccessDEC::allToAllTime( void* sendbuf, int sendcount , MPI_Datatype sendtype ,
                                   void* recvbuf, int recvcount , MPI_Datatype recvtype )
   {
-    int sts ;
+    int sts = 0;
     int target ;
     int sendoffset = 0 ;
     int SendTimeRequestId ;
@@ -632,7 +632,7 @@ namespace MEDCoupling
                                    void* recvbuf, int* recvcounts, int* rdispls,
                                    MPI_Datatype recvtype )
   {
-    int sts ;
+    int sts = 0;
     int target ;
     int SendTimeRequestId ;
     int SendDataRequestId ;
diff --git a/src/ParaMEDMEMTest/ParaMEDMEMTest_InterpKernelDEC.cxx b/src/ParaMEDMEMTest/ParaMEDMEMTest_InterpKernelDEC.cxx
index 0c4ecab12..9267c6621 100644
--- a/src/ParaMEDMEMTest/ParaMEDMEMTest_InterpKernelDEC.cxx
+++ b/src/ParaMEDMEMTest/ParaMEDMEMTest_InterpKernelDEC.cxx
@@ -492,10 +492,10 @@ void ParaMEDMEMTest::testInterpKernelDEC_2D_(const char *srcMeth, const char *ta
 
   MEDCoupling::InterpKernelDEC dec (*source_group,*target_group);
 
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::ParaMESH* paramesh;
-  MEDCoupling::ParaFIELD* parafield;
-  ICoCo::MEDField* icocofield ;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::ParaMESH* paramesh = nullptr;
+  MEDCoupling::ParaFIELD* parafield = nullptr;
+  ICoCo::MEDField* icocofield = nullptr;
   
   string filename_xml1              = "square1_split";
   string filename_xml2              = "square2_split";
@@ -700,8 +700,8 @@ void ParaMEDMEMTest::testInterpKernelDEC2_2D_(const char *srcMeth, const char *t
 
   MEDCoupling::InterpKernelDEC dec (*source_group,*target_group);
 
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::MEDCouplingFieldDouble* mcfield;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::MEDCouplingFieldDouble* mcfield = nullptr;
   
   string filename_xml1              = "square1_split";
   string filename_xml2              = "square2_split";
@@ -859,10 +859,10 @@ void ParaMEDMEMTest::testInterpKernelDEC_3D_(const char *srcMeth, const char *ta
 
   MEDCoupling::InterpKernelDEC dec (*source_group,*target_group);
 
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::ParaMESH* paramesh;
-  MEDCoupling::ParaFIELD* parafield;
-  ICoCo::MEDField* icocofield ;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::ParaMESH* paramesh = nullptr;
+  MEDCoupling::ParaFIELD* parafield = nullptr;
+  ICoCo::MEDField* icocofield = nullptr;
   
   char * tmp_dir_c                    = getenv("TMP");
   string tmp_dir;
@@ -2116,11 +2116,10 @@ void ParaMEDMEMTest::testAsynchronousInterpKernelDEC_2D(double dtA, double tmaxA
 
   MEDCoupling::InterpKernelDEC dec (*source_group,*target_group);
   
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::ParaMESH* paramesh;
-  MEDCoupling::ParaFIELD* parafield;
-  
-  ICoCo::MEDField* icocofield ;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::ParaMESH* paramesh = nullptr;
+  MEDCoupling::ParaFIELD* parafield = nullptr;
+  ICoCo::MEDField* icocofield = nullptr;
 
   char * tmp_dir_c                    = getenv("TMP");
   string tmp_dir;
diff --git a/src/ParaMEDMEMTest/ParaMEDMEMTest_StructuredCoincidentDEC.cxx b/src/ParaMEDMEMTest/ParaMEDMEMTest_StructuredCoincidentDEC.cxx
index 1f547230a..042d45b94 100644
--- a/src/ParaMEDMEMTest/ParaMEDMEMTest_StructuredCoincidentDEC.cxx
+++ b/src/ParaMEDMEMTest/ParaMEDMEMTest_StructuredCoincidentDEC.cxx
@@ -72,9 +72,9 @@ void ParaMEDMEMTest::testStructuredCoincidentDEC() {
   MEDCoupling::MPIProcessorGroup target_group(interface,3,size-1);
   MEDCoupling::MPIProcessorGroup source_group (interface,0,2);
 
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::ParaMESH* paramesh;
-  MEDCoupling::ParaFIELD* parafield;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::ParaMESH* paramesh = nullptr;
+  MEDCoupling::ParaFIELD* parafield = nullptr;
 
   string filename_xml1 = INTERP_TEST::getResourceFile("square1_split");
   string filename_2    = INTERP_TEST::getResourceFile("square1.med");
diff --git a/src/ParaMEDMEMTest/test_MPI_Access_Cancel.cxx b/src/ParaMEDMEMTest/test_MPI_Access_Cancel.cxx
index e1290eda3..99ba5c5ca 100644
--- a/src/ParaMEDMEMTest/test_MPI_Access_Cancel.cxx
+++ b/src/ParaMEDMEMTest/test_MPI_Access_Cancel.cxx
@@ -82,7 +82,7 @@ void MPIAccessTest::test_MPI_Access_Cancel() {
   int intsendbuf[5] ;
   double doublesendbuf[10] ;
   int RequestId[10] ;
-  int sts ;
+  int sts = 0;
   int i , j ;
   for ( j = 0 ; j < 3 ; j++ ) {
      for ( i = 0 ; i < 10 ; i++ ) {
diff --git a/src/ParaMEDMEMTest/test_MPI_Access_IProbe.cxx b/src/ParaMEDMEMTest/test_MPI_Access_IProbe.cxx
index 5476f8b6e..46c391ee1 100644
--- a/src/ParaMEDMEMTest/test_MPI_Access_IProbe.cxx
+++ b/src/ParaMEDMEMTest/test_MPI_Access_IProbe.cxx
@@ -81,7 +81,7 @@ void MPIAccessTest::test_MPI_Access_IProbe() {
   int target = 1 - myrank ;
   int sendbuf[10] ;
   int RequestId[10] ;
-  int sts ;
+  int sts = 0;
   int i ;
   for ( i = 0 ; i < 10 ; i++ ) {
      if ( myrank == 0 ) {
diff --git a/src/ParaMEDMEMTest/test_MPI_Access_ISend_IRecv_Length_1.cxx b/src/ParaMEDMEMTest/test_MPI_Access_ISend_IRecv_Length_1.cxx
index 818db992d..674d47e82 100644
--- a/src/ParaMEDMEMTest/test_MPI_Access_ISend_IRecv_Length_1.cxx
+++ b/src/ParaMEDMEMTest/test_MPI_Access_ISend_IRecv_Length_1.cxx
@@ -95,8 +95,8 @@ void MPIAccessTest::test_MPI_Access_ISend_IRecv_Length_1() {
     int j ;
     for (j = 1 ; j <= i ; j++) {
       int source ;
-      MPI_Datatype datatype ;
-      int outcount ;
+      MPI_Datatype datatype = 0;
+      int outcount = 0;
       int flag ;
       if ( myrank == 0 ) {
         mpi_access.test( SendRequestId[j], flag ) ;
diff --git a/src/ParaMEDMEMTest/test_MPI_Access_Time_0.cxx b/src/ParaMEDMEMTest/test_MPI_Access_Time_0.cxx
index f7f253b19..e168a5928 100644
--- a/src/ParaMEDMEMTest/test_MPI_Access_Time_0.cxx
+++ b/src/ParaMEDMEMTest/test_MPI_Access_Time_0.cxx
@@ -112,7 +112,7 @@ void MPIAccessTest::test_MPI_Access_Time_0() {
   int RecvTimeRequestId[maxreq] ;
   int SendRequestId[maxreq] ;
   int RecvRequestId[maxreq] ;
-  int sts ;
+  int sts = 0;
   int sendbuf[maxreq] ;
   int recvbuf[maxreq] ;
   MEDCoupling::TimeMessage aSendTimeMsg[maxreq] ;
diff --git a/src/ParaMEDMEMTest/test_perf.cxx b/src/ParaMEDMEMTest/test_perf.cxx
index 86d7534af..ee5b273eb 100644
--- a/src/ParaMEDMEMTest/test_perf.cxx
+++ b/src/ParaMEDMEMTest/test_perf.cxx
@@ -146,10 +146,10 @@ void testInterpKernelDEC_2D(const string& filename_xml1, const string& meshname1
   else
     dec.setIntersectionType(INTERP_KERNEL::Convex);
 
-  MEDCoupling::MEDCouplingUMesh* mesh;
-  MEDCoupling::ParaMESH* paramesh;
-  MEDCoupling::ParaFIELD* parafield;
-  ICoCo::MEDField* icocofield ;
+  MEDCoupling::MEDCouplingUMesh* mesh = nullptr;
+  MEDCoupling::ParaMESH* paramesh = nullptr;
+  MEDCoupling::ParaFIELD* parafield = nullptr;
+  ICoCo::MEDField* icocofield = nullptr;
   
   // To remove tmp files from disk
   ParaMEDMEMTest_TmpFilesRemover aRemover;