initialize variables with a default value 02/7502/2
Allan CORNET [Tue, 12 Jun 2012 07:50:45 +0000 (09:50 +0200)]
Change-Id: I64df134fc7c5360cd54e771606f3e5f9a1f7a096

22 files changed:
scilab/modules/graphic_objects/src/cpp/ColorComputer.cpp
scilab/modules/graphic_objects/src/cpp/DataLoader.cpp
scilab/modules/graphic_objects/src/cpp/DecompositionUtils.cpp
scilab/modules/graphic_objects/src/cpp/Fac3DColorComputer.cpp
scilab/modules/graphic_objects/src/cpp/Fac3DDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/MatPlotDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/NgonGeneralData.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridData.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridDataDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridGrayplotDataDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridMatplotData.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridMatplotDataDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/NgonPolylineData.cpp
scilab/modules/graphic_objects/src/cpp/Plot3DDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/PolylineDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/ScilabView.cpp
scilab/modules/graphic_objects/src/cpp/TriangleMeshData.cpp
scilab/modules/graphic_objects/src/cpp/TriangleMeshFecData.cpp
scilab/modules/graphic_objects/src/cpp/TriangleMeshFecDataDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/Triangulator.cpp
scilab/modules/graphic_objects/src/cpp/dataModel.cpp
scilab/modules/graphic_objects/src/cpp/setGraphicObjectProperty.cpp

index 180486c..d3a81fe 100644 (file)
@@ -23,8 +23,8 @@ extern "C"
 
 void ColorComputer::getColor(double s, double smin, double srange, double indexOffset, double* colormap, int colormapSize, float* returnedColor)
 {
-    double value;
-    int index;
+    double value = 0.;
+    int index = 0;
 
     if (!DecompositionUtils::isANumber(s))
     {
@@ -56,8 +56,8 @@ void ColorComputer::getColor(double s, double smin, double srange, double indexO
 
 void ColorComputer::getColor(double s, double smin, double srange, double indexOffset, double* colormap, int minIndex, int maxIndex, int colormapSize, float* returnedColor)
 {
-    double value;
-    int index;
+    double value = 0.;
+    int index = 0;
 
     if (!DecompositionUtils::isANumber(s))
     {
@@ -89,8 +89,8 @@ void ColorComputer::getColor(double s, double smin, double srange, double indexO
 
 double ColorComputer::getIndex(double s, double smin, double srange, double indexOffset, int minIndex, int maxIndex)
 {
-    double value;
-    double index;
+    double value = 0.;
+    double index = 0.;
 
     if (!DecompositionUtils::isANumber(s))
     {
@@ -120,7 +120,7 @@ double ColorComputer::getIndex(double s, double smin, double srange, double inde
 
 void ColorComputer::getDirectColor(double s, double* colormap, int colormapSize, float* returnedColor)
 {
-    int index;
+    int index = 0;
 
     if (s <= (double) BLACK_LOWER_INDEX)
     {
@@ -153,7 +153,7 @@ void ColorComputer::getDirectColor(double s, double* colormap, int colormapSize,
 
 double ColorComputer::getDirectIndex(double s, int colormapSize)
 {
-    double index;
+    double index = 0.;
 
     if (s <= (double) WHITE_LOWER_INDEX)
     {
@@ -222,7 +222,7 @@ double ColorComputer::getClampedDirectIndex(double s, int colormapSize)
 
 void ColorComputer::getClampedDirectColor(double s, double* colormap, int colormapSize, float* returnedColor)
 {
-    int index;
+    int index = 0;
 
     if (!DecompositionUtils::isANumber(s))
     {
index 471b539..51bbe3f 100644 (file)
@@ -34,7 +34,7 @@ extern "C"
 
 int getTextureWidth(char* id)
 {
-  char* type;
+  char* type = NULL;
   getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
   if (strcmp(type, __GO_MATPLOT__) == 0)
   {
@@ -48,7 +48,7 @@ int getTextureWidth(char* id)
 
 int getTextureHeight(char* id)
 {
-  char* type;
+  char* type = NULL;
   getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
   if (strcmp(type, __GO_MATPLOT__) == 0)
   {
@@ -62,7 +62,7 @@ int getTextureHeight(char* id)
 
 int fillTextureData(char* id, unsigned char* buffer, int bufferLength)
 {
-  char* type;
+  char* type = NULL;
   getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
   if (strcmp(type, __GO_MATPLOT__) == 0)
   {
@@ -76,7 +76,7 @@ int fillTextureData(char* id, unsigned char* buffer, int bufferLength)
 
 int fillSubTextureData(char* id, unsigned char* buffer, int bufferLength, int x, int y, int width, int height)
 {
-  char* type;
+  char* type = NULL;
   getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
   if (strcmp(type, __GO_MATPLOT__) == 0)
   {
@@ -90,7 +90,7 @@ int fillSubTextureData(char* id, unsigned char* buffer, int bufferLength, int x,
 
 int getDataSize(char* id)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -127,7 +127,7 @@ int getDataSize(char* id)
 
 void fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -159,7 +159,7 @@ void fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, i
 
 void fillTextureCoordinates(char* id, float* BUFF, int bufferLength)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -179,7 +179,7 @@ void fillTextureCoordinates(char* id, float* BUFF, int bufferLength)
 
 void fillColors(char* id, float* BUFF, int bufferLength, int elementsSize)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -208,7 +208,7 @@ void fillColors(char* id, float* BUFF, int bufferLength, int elementsSize)
 
 int getIndicesSize(char* id)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -243,7 +243,7 @@ int getIndicesSize(char* id)
 
 int fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
@@ -277,7 +277,7 @@ int fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 
 int getWireIndicesSize(char* id)
 {
-    char* type;
+    char* type = NULL;
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
     if (strcmp(type, __GO_FAC3D__) == 0)
@@ -302,7 +302,7 @@ int getWireIndicesSize(char* id)
 
 int fillWireIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    char* type;
+    char* type = NULL;
 
     getGraphicObjectProperty(id, __GO_TYPE__, jni_string, (void**) &type);
 
index 7d407d1..1e71932 100644 (file)
@@ -161,13 +161,13 @@ void DecompositionUtils::getDecomposedQuadTriangleIndices(double vertices[4][3],
     double mo0[3];
     double mo1[3];
 
-    double nmo0;
-    double nmo1;
+    double nmo0 = 0.;
+    double nmo1 = 0.;
 
-    double dot0;
-    double dot1;
+    double dot0 = 0.;
+    double dot1 = 0.;
 
-    double denom;
+    double denom = 0.;
 
     /*
      * The input vertices are given in counter-clockwise order, from v0 to v3.
index fff6618..c295768 100644 (file)
@@ -254,8 +254,8 @@ int Fac3DColorComputer::isFacetColorValid(int facetIndex)
 
 double Fac3DColorComputer::computeAverageValue(int facetIndex)
 {
-    double averageValue;
-    int i;
+    double averageValue = 0.;
+    int i = 0;
 
     averageValue = 0.0;
 
@@ -271,11 +271,11 @@ double Fac3DColorComputer::computeAverageValue(int facetIndex)
 
 void Fac3DColorComputer::computeMinMaxValues(void)
 {
-    double maxDouble;
-    double tmpValueMin;
-    double tmpValueMax;
+    double maxDouble = 0.;
+    double tmpValueMin = 0.;
+    double tmpValueMax = 0.;
 
-    int i;
+    int i = 0;
 
     maxDouble = DecompositionUtils::getMaxDoubleValue();
 
@@ -293,7 +293,7 @@ void Fac3DColorComputer::computeMinMaxValues(void)
 
 void Fac3DColorComputer::computeFacetMinMax(int facetIndex, double* currentMin, double* currentMax)
 {
-    double value;
+    double value = 0.;
 
     for (int i = 0; i < numColorsPerFacet; i++)
     {
index a246b1c..e6d1fc9 100644 (file)
@@ -39,11 +39,11 @@ int Fac3DDecomposer::getDataSize(char* id)
 
 void Fac3DDecomposer::fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
-    int i;
+    int i = 0;
 
     int numVerticesPerGon = 0;
     int* piNumVerticesPerGon = &numVerticesPerGon;
@@ -89,14 +89,14 @@ void Fac3DDecomposer::fillVertices(char* id, float* buffer, int bufferLength, in
 
 void Fac3DDecomposer::fillTextureCoordinates(char* id, float* buffer, int bufferLength)
 {
-    char* parentFigure;
-    char* parent;
+    char* parentFigure = NULL;
+    char* parent = NULL;
 
-    double* colors;
-    double* colormap;
-    double* z;
+    double* colors = NULL;
+    double* colormap = NULL;
+    double* z = NULL;
 
-    double color;
+    double color = 0.;
 
     int numVerticesPerGon = 0;
     int* piNumVerticesPerGon = &numVerticesPerGon;
@@ -195,15 +195,15 @@ void Fac3DDecomposer::fillTextureCoordinates(char* id, float* buffer, int buffer
 void Fac3DDecomposer::fillNormalizedZColorsTextureCoordinates(float* buffer, int bufferLength, double* colormap, int colormapSize,
     double* z, int numGons, int numVerticesPerGon)
 {
-    double zavg;
-    double zMin;
-    double zMax;
-    double zRange;
-    double minDoubleValue;
-    float index;
-
-    int i;
-    int j;
+    double zavg = 0.;
+    double zMin = 0.;
+    double zMax = 0.;
+    double zRange = 0.;
+    double minDoubleValue = 0.;
+    float index = 0.;
+
+    int i = 0;
+    int j = 0;
     int bufferOffset = 0;
 
     computeMinMaxValues(z, numGons*numVerticesPerGon, numGons, numVerticesPerGon, ALL_VALUES, &zMin, &zMax);
@@ -240,12 +240,11 @@ void Fac3DDecomposer::fillNormalizedZColorsTextureCoordinates(float* buffer, int
 void Fac3DDecomposer::fillConstantColorsTextureCoordinates(float* buffer, int bufferLength, double* colormap, int colormapSize,
     double colorValue, int numGons, int numVerticesPerGon)
 {
-    int i;
     int bufferOffset = 0;
 
     double index = (ColorComputer::getClampedDirectIndex(colorValue - 1.0, colormapSize) + 2.0 + COLOR_TEXTURE_OFFSET) / (float) (colormapSize + 2);
 
-    for (i = 0; i < numGons*numVerticesPerGon; i++)
+    for (int i = 0; i < numGons*numVerticesPerGon; i++)
     {
         buffer[bufferOffset++] = (float)index;
         buffer[bufferOffset++] = 0;
@@ -264,8 +263,6 @@ void Fac3DDecomposer::fillDataColorsTextureCoordinates(float* buffer, int buffer
     double colorTextureOffset = 0.;
     double index = 0.;
 
-    int i = 0;
-    int j = 0;
     int bufferOffset = 0;
 
     int numColors = 0;
@@ -296,9 +293,9 @@ void Fac3DDecomposer::fillDataColorsTextureCoordinates(float* buffer, int buffer
      */
     colorTextureOffset = COLOR_TEXTURE_OFFSET;
 
-    for (i = 0; i < numGons; i++)
+    for (int i = 0; i < numGons; i++)
     {
-        for (j = 0; j < numVerticesPerGon; j++)
+        for (int j = 0; j < numVerticesPerGon; j++)
         {
             color = colorComputer.getOutputFacetColor(i, j);
 
@@ -323,12 +320,9 @@ void Fac3DDecomposer::fillDataColorsTextureCoordinates(float* buffer, int buffer
 
 double Fac3DDecomposer::computeAverageValue(double* values, int numVertices)
 {
-    double averageValue;
-    int i;
-
-    averageValue = 0.0;
+    double averageValue = 0.0;
 
-    for (i = 0; i < numVertices; i++)
+    for (int i = 0; i < numVertices; i++)
     {
         averageValue += values[i];
     }
@@ -341,19 +335,13 @@ double Fac3DDecomposer::computeAverageValue(double* values, int numVertices)
 void Fac3DDecomposer::computeMinMaxValues(double* values, int numValues, int numGons, int numVerticesPerGon, int minMaxComputation,
     double* valueMin, double* valueMax)
 {
-    double maxDouble;
-    double tmpValueMin;
-    double tmpValueMax;
-    double value;
-
-    int i;
-    int numIterations;
-
-    maxDouble = DecompositionUtils::getMaxDoubleValue();
-
-    tmpValueMin = maxDouble;
-    tmpValueMax = -maxDouble;
+    double maxDouble = DecompositionUtils::getMaxDoubleValue();
+    double tmpValueMin = maxDouble;
+    double tmpValueMax = -maxDouble;
+    double value = 0.;
 
+    int numIterations = 0;
+    
     if (minMaxComputation != ALL_VALUES)
     {
         numIterations = numGons;
@@ -363,7 +351,7 @@ void Fac3DDecomposer::computeMinMaxValues(double* values, int numValues, int num
         numIterations = numValues;
     }
 
-    for (i = 0; i < numIterations; i++)
+    for (int i = 0; i < numIterations; i++)
     {
         if (minMaxComputation == FIRST_VERTEX_VALUE)
         {
@@ -420,15 +408,15 @@ int Fac3DDecomposer::getIndicesSize(char* id)
  */
 int Fac3DDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
-    double xc;
-    double yc;
-    double zc;
+    double xc = 0.;
+    double yc = 0.;
+    double zc = 0.;
 
-    double* colors;
+    double* colors = NULL;
 
     int numVerticesPerGon = 0;
     int* piNumVerticesPerGon = &numVerticesPerGon;
@@ -444,9 +432,6 @@ int Fac3DDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int lo
     int dataMapping =  0;
     int* piDataMapping = &dataMapping;
 
-    int i;
-    int j;
-
     int bufferOffset = 0;
     int vertexOffset = 0;
 
@@ -474,11 +459,11 @@ int Fac3DDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int lo
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Y__, jni_double_vector, (void**) &y);
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Z__, jni_double_vector, (void**) &z);
 
-    for (i = 0; i < numGons; i++)
+    for (int i = 0; i < numGons; i++)
     {
         int isValid = 1;
 
-        for (j = 0; j < numVerticesPerGon; j++)
+        for (int j = 0; j < numVerticesPerGon; j++)
         {
             xc = x[vertexOffset+j];
             yc = y[vertexOffset+j];
@@ -498,7 +483,7 @@ int Fac3DDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int lo
         }
 
         /* Performs a fan decomposition, vertices are ordered counter-clockwise. */
-        for (j = 0; j < numVerticesPerGon-2; j++)
+        for (int j = 0; j < numVerticesPerGon-2; j++)
         {
             buffer[bufferOffset] = vertexOffset;
             buffer[bufferOffset +1] = vertexOffset + j+2;
@@ -528,22 +513,19 @@ int Fac3DDecomposer::getWireIndicesSize(char* id)
 
 int Fac3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
-    double xc;
-    double yc;
-    double zc;
+    double xc = 0.;
+    double yc = 0.;
+    double zc = 0.;
 
     int numVerticesPerGon = 0;
     int* piNumVerticesPerGon = &numVerticesPerGon;
     int numGons = 0;
     int* piNumGons = &numGons;
 
-    int i;
-    int j;
-
     int bufferOffset = 0;
     int vertexOffset = 0;
 
@@ -560,11 +542,11 @@ int Fac3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, in
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Y__, jni_double_vector, (void**) &y);
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Z__, jni_double_vector, (void**) &z);
 
-    for (i = 0; i < numGons; i++)
+    for (int i = 0; i < numGons; i++)
     {
         int isValid = 1;
 
-        for (j = 0; j < numVerticesPerGon; j++)
+        for (int j = 0; j < numVerticesPerGon; j++)
         {
             xc = x[vertexOffset+j];
             yc = y[vertexOffset+j];
@@ -583,7 +565,7 @@ int Fac3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, in
             continue;
         }
 
-        for (j = 0; j < numVerticesPerGon; j++)
+        for (int j = 0; j < numVerticesPerGon; j++)
         {
             buffer[bufferOffset] = vertexOffset + j;
             buffer[bufferOffset+1] = vertexOffset + (j+1) % numVerticesPerGon;
index 6dc5408..4172cf0 100644 (file)
@@ -23,7 +23,7 @@ extern "C" {
 
 int MatPlotDecomposer::getTextureWidth(char* id)
 {
-  int width;
+  int width = 0;
   int* piWidth = &width;
   getGraphicObjectProperty(id, __GO_DATA_MODEL_NUM_X__, jni_int, (void**) &piWidth);
   return width - 1;
@@ -32,7 +32,7 @@ int MatPlotDecomposer::getTextureWidth(char* id)
 
 int MatPlotDecomposer::getTextureHeight(char* id)
 {
-  int height;
+  int height = 0;
   int* piHeight = &height;
   getGraphicObjectProperty(id, __GO_DATA_MODEL_NUM_Y__, jni_int, (void**) &piHeight);
   return height - 1;
@@ -41,7 +41,7 @@ int MatPlotDecomposer::getTextureHeight(char* id)
 
 int MatPlotDecomposer::fillTextureData(char* id, unsigned char* buffer, int bufferLength)
 {
-  double* value;
+  double* value = NULL;
   getGraphicObjectProperty(id, __GO_DATA_MODEL_Z__, jni_double_vector, (void**) &value);
   int textureWidth = getTextureWidth(id);
   int textureHeight = getTextureHeight(id);
@@ -49,9 +49,9 @@ int MatPlotDecomposer::fillTextureData(char* id, unsigned char* buffer, int buff
   if (dataSize * 4 == bufferLength)
   {
 
-    char* parentFigure;
-    double* colormap;
-    int colormapSize;
+    char* parentFigure = NULL;
+    double* colormap = NULL;
+    int colormapSize = 0;
     int* piColormapSize = &colormapSize;
 
     getGraphicObjectProperty(id, __GO_PARENT_FIGURE__, jni_string, (void**) &parentFigure);
@@ -78,13 +78,13 @@ int MatPlotDecomposer::fillTextureData(char* id, unsigned char* buffer, int buff
 
 int MatPlotDecomposer::fillTextureData(char* id, unsigned char* buffer, int bufferLength, int x, int y, int width, int height)
 {
-  double* value;
+  double* value = NULL;
   getGraphicObjectProperty(id, __GO_DATA_MODEL_Z__, jni_double_vector, (void**) &value);
   if (width * height * 4 == bufferLength)
   {
-    char* parentFigure;
-    double* colormap;
-    int colormapSize;
+    char* parentFigure = NULL;
+    double* colormap = NULL;
+    int colormapSize = 0;
     int* piColormapSize = &colormapSize;
     getGraphicObjectProperty(id, __GO_PARENT_FIGURE__, jni_string, (void**) &parentFigure);
     getGraphicObjectProperty(parentFigure, __GO_COLORMAP__, jni_double_vector, (void**) &colormap);
index edf14d0..5435f9b 100644 (file)
@@ -196,7 +196,7 @@ void NgonGeneralData::setData(double const* data, int numElements)
 
 void NgonGeneralData::setDataX(double const* data, int numElements)
 {
-    double* xCoordinates;
+    double* xCoordinates = NULL;
 
     xCoordinates = coordinates;
 
@@ -222,7 +222,7 @@ void NgonGeneralData::setDataY(double const* data, int numElements)
 
 void NgonGeneralData::setDataZ(double const* data, int numElements)
 {
-    double* zCoordinates;
+    double* zCoordinates = NULL;
 
     zCoordinates = &coordinates[2 * numGons * numVerticesPerGon];
 
@@ -242,9 +242,7 @@ int NgonGeneralData::setNumElementsArray(int const* numElementsArray)
 {
     double* newCoordinates = NULL;
     double* newColorValues = NULL;
-    int result;
-
-    result = 1;
+    int result = 1;
 
     /* Test whether the number of colors is valid */
     if ((numElementsArray[2] != numElementsArray[0]*numElementsArray[1]) &&
index 6834c6c..5a80a6c 100644 (file)
@@ -187,12 +187,12 @@ void NgonGridData::getDataProperty(int property, void **_pvData)
 
 int NgonGridData::setGridSize(int const* gridSize)
 {
-    int newXSize;
-    int newYSize;
-    int xModified;
-    int yModified;
-    int zModified;
-    int result;
+    int newXSize = 0;
+    int newYSize = 0;
+    int xModified = 0;
+    int yModified = 0;
+    int zModified = 0;
+    int result = 0;
 
     double* newXCoordinates = NULL;
     double* newYCoordinates = NULL;
index c188b23..45a95cf 100644 (file)
@@ -43,9 +43,9 @@ int NgonGridDataDecomposer::getDataSize(char* id)
 
 void NgonGridDataDecomposer::fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double* x;
-    double* y;
-    double zShift;
+    double* x = NULL;
+    double* y = NULL;
+    double zShift = 0.;
     double* pdZShift = &zShift;
 
     int numX = 0;
@@ -68,17 +68,14 @@ void NgonGridDataDecomposer::fillVertices(char* id, float* buffer, int bufferLen
 void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask,
     double* x, double* y, double* z, int numX, int numY)
 {
-    double xi;
-    double yj;
-    double zij;
+    double xi = 0.;
+    double yj = 0.;
+    double zij = 0.;
 
-    int i;
-    int j;
-
-    int bufferOffset;
+    int bufferOffset = 0;
 
 #if PER_VERTEX_VALUES
-    for (j = 0; j < numY; j++)
+    for (int j = 0; j < numY; j++)
     {
         yj = y[j];
 
@@ -90,7 +87,7 @@ void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, i
             }
         }
 
-        for (i = 0; i < numX; i++)
+        for (int i = 0; i < numX; i++)
         {
             xi = x[i];
             bufferOffset = elementsSize*(numX*j + i);
@@ -125,12 +122,12 @@ void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, i
     }
 #else
 
-    double yjp1;
-    double xip1;
+    double yjp1 = 0.;
+    double xip1 = 0.;
 
     bufferOffset = 0;
 
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
         double ycoords[4];
         int yindices[4];
@@ -157,11 +154,10 @@ void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, i
         yindices[2] = j+1;
         yindices[3] = j+1;
 
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             double xcoords[4];
             int xindices[4];
-            int k;
 
             xi = x[i];
             xip1 = x[i+1];
@@ -182,7 +178,7 @@ void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, i
             xindices[2] = i;
             xindices[3] = i+1;
 
-            for (k = 0; k < 4; k++)
+            for (int k = 0; k < 4; k++)
             {
                 if (coordinateMask & 0x1)
                 {
@@ -220,13 +216,10 @@ void NgonGridDataDecomposer::fillGridVertices(float* buffer, int bufferLength, i
 void NgonGridDataDecomposer::fillNormalizedZGridColors(float* buffer, int bufferLength, int elementsSize, double* colormap, int colormapSize,
     double* z, int numX, int numY)
 {
-    double zMin;
-    double zMax;
-    double zRange;
-    double minDoubleValue;
-
-    int i;
-    int j;
+    double zMin = 0.;
+    double zMax = 0.;
+    double zRange = 0.;
+    double minDoubleValue = 0.;
 
     int bufferOffset = 0;
 
@@ -245,9 +238,9 @@ void NgonGridDataDecomposer::fillNormalizedZGridColors(float* buffer, int buffer
     }
 
 #if PER_VERTEX_VALUES
-    for (j = 0; j < numY; j++)
+    for (int j = 0; j < numY; j++)
     {
-        for (i = 0; i < numX; i++)
+        for (int i = 0; i < numX; i++)
         {
             int currentPointIndex = getPointIndex(numX, numY, i, j);
 
@@ -262,9 +255,9 @@ void NgonGridDataDecomposer::fillNormalizedZGridColors(float* buffer, int buffer
         }
     }
 #else
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             double avgz;
             float facetColor[3];
@@ -285,16 +278,13 @@ void NgonGridDataDecomposer::fillNormalizedZGridColors(float* buffer, int buffer
 void NgonGridDataDecomposer::fillDirectGridColors(float* buffer, int bufferLength, int elementsSize, double* colormap, int colormapSize,
     double* z, int numX, int numY)
 {
-    int i;
-    int j;
-
-    int currentPointIndex;
+    int currentPointIndex = 0;
     int bufferOffset = 0;
 
 #if PER_VERTEX_VALUES
-    for (j = 0; j < numY; j++)
+    for (int j = 0; j < numY; j++)
     {
-        for (i = 0; i < numX; i++)
+        for (int i = 0; i < numX; i++)
         {
             currentPointIndex = getPointIndex(numX, numY, i, j);
 
@@ -309,9 +299,9 @@ void NgonGridDataDecomposer::fillDirectGridColors(float* buffer, int bufferLengt
         }
     }
 #else
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             float facetColor[3];
 
@@ -355,7 +345,7 @@ double NgonGridDataDecomposer::getZCoordinate(double* z, int numX, int numY, int
 
 double NgonGridDataDecomposer::getZCoordinate(double* z, int numX, int numY, int i, int j, int logUsed)
 {
-    double zij;
+    double zij = 0.;
 
     if (logUsed)
     {
@@ -396,16 +386,16 @@ int NgonGridDataDecomposer::getIndicesSize(char* id)
 
 int NgonGridDataDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
     int numX = 0;
     int* piNumX = &numX;
     int numY = 0;
     int* piNumY = &numY;
 
-    int numberIndices;
+    int numberIndices = 0;
 
     NgonGridDataDecomposer* decomposer = get();
 
@@ -432,25 +422,23 @@ int NgonGridDataDecomposer::fillIndices(char* id, int* buffer, int bufferLength,
  */
 int NgonGridDataDecomposer::fillTriangleIndices(int* buffer, int bufferLength, int logMask, double* x, double* y, double* z, double* values, int numX, int numY)
 {
-    int i;
-    int j;
     int bufferOffset = 0;
 
-    int currentRowValid;
-    int nextRowValid;
+    int currentRowValid = 0;
+    int nextRowValid = 0;
 
-    int currentColumnValid;
-    int nextColumnValid;
+    int currentColumnValid = 0;
+    int nextColumnValid = 0;
 
-    int currentEdgeValid;
-    int nextEdgeValid;
+    int currentEdgeValid = 0;
+    int nextEdgeValid = 0;
 
-    int currentFacetValid;
+    int currentFacetValid = 0;
 
-    int ij;
-    int ip1j;
-    int ip1jp1;
-    int ijp1;
+    int ij = 0;
+    int ip1j = 0;
+    int ip1jp1 = 0;
+    int ijp1 = 0;
 
     /* 0 indices if less than 2 points along either dimension */
     if (numX < 2 || numY < 2)
@@ -467,7 +455,7 @@ int NgonGridDataDecomposer::fillTriangleIndices(int* buffer, int bufferLength, i
     }
 
     /* To do: optimize */
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
         nextRowValid = DecompositionUtils::isValid(y[j+1]);
 
@@ -498,7 +486,7 @@ int NgonGridDataDecomposer::fillTriangleIndices(int* buffer, int bufferLength, i
 
         currentEdgeValid = isFacetEdgeValid(z, values, numX, numY, 0, j, logMask & 0x4);
 
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             nextColumnValid = DecompositionUtils::isValid(x[i+1]);
 
@@ -584,11 +572,11 @@ int NgonGridDataDecomposer::isFacetValid(double* z, double* values, int numX, in
 
 int NgonGridDataDecomposer::isFacetEdgeValid(double* z, double* values, int numX, int numY, int i, int j, int logUsed)
 {
-    double zij;
-    double zijp1;
+    double zij = 0.;
+    double zijp1 = 0.;
 
-    int lowerZValid;
-    int upperZValid;
+    int lowerZValid = 0;
+    int upperZValid = 0;
 
     zij = getZCoordinate(z, numX, numY, i, j);
     zijp1 = getZCoordinate(z, numX, numY, i, j+1);
@@ -619,23 +607,21 @@ int NgonGridDataDecomposer::getPointIndex(int numX, int numY, int i, int j)
 
 void NgonGridDataDecomposer::computeMinMaxZValues(double* z, int numX, int numY, double* zMin, double* zMax)
 {
-    double maxDouble;
-    double tmpZMin;
-    double tmpZMax;
-    double zi;
+    double maxDouble = 0;
+    double tmpZMin = 0;
+    double tmpZMax = 0;
+    double zi = 0;
 
-    int i;
-    int j;
-    int currentPoint;
+    int currentPoint = 0;
 
     maxDouble = DecompositionUtils::getMaxDoubleValue();
 
     tmpZMin = maxDouble;
     tmpZMax = -maxDouble;
 
-    for (j = 0; j < numY; j++)
+    for (int j = 0; j < numY; j++)
     {
-        for (i = 0; i < numX; i++)
+        for (int i = 0; i < numX; i++)
         {
             currentPoint = getPointIndex(numX, numY, i, j);
             zi = z[currentPoint];
@@ -661,8 +647,8 @@ void NgonGridDataDecomposer::computeMinMaxZValues(double* z, int numX, int numY,
 
 double NgonGridDataDecomposer::computeFacetAverageZValue(double* z, int numX, int numY, int i, int j)
 {
-    double avgz;
-    int pointIndex;
+    double avgz = 0.;
+    int pointIndex = 0;
 
     avgz = 0.0;
     pointIndex = getPointIndex(numX, numY, i, j);
@@ -681,9 +667,7 @@ double NgonGridDataDecomposer::computeFacetAverageZValue(double* z, int numX, in
 
 void NgonGridDataDecomposer::writeFacetColorToBuffer(float* buffer, int bufferOffset, float* color, int elementsSize)
 {
-    int k;
-
-    for (k = 0; k < 4; k++)
+    for (int k = 0; k < 4; k++)
     {
         buffer[bufferOffset] = color[0];
         buffer[bufferOffset +1] = color[1];
index fc6c00f..08e0e98 100644 (file)
@@ -30,11 +30,11 @@ NgonGridGrayplotDataDecomposer* NgonGridGrayplotDataDecomposer::decomposer = NUL
  */
 void NgonGridGrayplotDataDecomposer::fillColors(char* id, float* buffer, int bufferLength, int elementsSize)
 {
-    char* parentFigure;
-    char* parent;
+    char* parentFigure = NULL;
+    char* parent = NULL;
 
-    double* z;
-    double* colormap;
+    double* z = NULL;
+    double* colormap = NULL;
 
     int numX = 0;
     int* piNumX = &numX;
@@ -85,9 +85,9 @@ void NgonGridGrayplotDataDecomposer::fillColors(char* id, float* buffer, int buf
 
 int NgonGridGrayplotDataDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
     double zShift = 0.0;
     double* pdZShift = &zShift;
 
@@ -96,7 +96,7 @@ int NgonGridGrayplotDataDecomposer::fillIndices(char* id, int* buffer, int buffe
     int numY = 0;
     int* piNumY = &numY;
 
-    int numberIndices;
+    int numberIndices = 0;
 
     NgonGridGrayplotDataDecomposer* decomposer = get();
 
@@ -125,11 +125,11 @@ int NgonGridGrayplotDataDecomposer::fillIndices(char* id, int* buffer, int buffe
  */
 int NgonGridGrayplotDataDecomposer::isFacetEdgeValid(double* z, double* values, int numX, int numY, int i, int j, int logUsed)
 {
-    double zij;
-    double zijp1;
+    double zij = 0.;
+    double zijp1 = 0.;
 
-    int lowerZValid;
-    int upperZValid;
+    int lowerZValid = 0;
+    int upperZValid = 0;
 
     /* First, z-coordinate values are tested */
     zij = getZCoordinate(z, numX, numY, i, j, logUsed);
index 9cf73cb..9f50371 100644 (file)
@@ -102,13 +102,13 @@ void NgonGridMatplotData::getDataProperty(int property, void **_pvData)
  */
 int NgonGridMatplotData::setGridSize(int const* gridSize)
 {
-    int newXSize;
-    int newYSize;
-    int xModified;
-    int yModified;
-    int zModified;
-    int result;
-    int formerGridSize;
+    int newXSize = 0;
+    int newYSize = 0;
+    int xModified = 0;
+    int yModified = 0;
+    int zModified = 0;
+    int result = 0;
+    int formerGridSize = 0;
 
     double* newXCoordinates = NULL;
     double* newYCoordinates = NULL;
@@ -286,16 +286,14 @@ double* NgonGridMatplotData::getBounds(void)
 
 void NgonGridMatplotData::computeCoordinates(void)
 {
-    int i;
-
     if (type == 0)
     {
-        for (i = 0; i < xSize; i++)
+        for (int i = 0; i < xSize; i++)
         {
             xCoordinates[i] = 0.5 + (double) i;
         }
 
-        for (i = 0; i < ySize; i++)
+        for (int i = 0; i < ySize; i++)
         {
             yCoordinates[i] = 0.5 + (double) i;
         }
@@ -307,12 +305,10 @@ void NgonGridMatplotData::computeCoordinates(void)
     }
     else
     {
-        double min;
-        double max;
-        int numElements;
+        double min = boundingRectangle[0];
+        double max = boundingRectangle[1];
+        int numElements = 0;
 
-        min = boundingRectangle[0];
-        max = boundingRectangle[1];
 
         if (xSize == 1)
         {
@@ -323,7 +319,7 @@ void NgonGridMatplotData::computeCoordinates(void)
             numElements = xSize - 1;
         }
 
-        for (i = 0; i < xSize; i++)
+        for (int i = 0; i < xSize; i++)
         {
             xCoordinates[i] = min + (double) i * (max - min) / (double) numElements;
         }
@@ -340,7 +336,7 @@ void NgonGridMatplotData::computeCoordinates(void)
             numElements = ySize - 1;
         }
 
-        for (i = 0; i < ySize; i++)
+        for (int i = 0; i < ySize; i++)
         {
             yCoordinates[i] = min + (double) i * (max - min) / (double) numElements;
         }
index e5b42fc..979c4a4 100644 (file)
@@ -30,7 +30,7 @@ void NgonGridMatplotDataDecomposer::fillVertices(char* id, float* buffer, int bu
 {
     double* matplotScale = NULL;
     double* matplotTranslate = NULL;
-    double zShift;
+    double zShift = 0.;
     double* pdZShift = &zShift;
 
     double xTrans[2];
@@ -70,19 +70,16 @@ void NgonGridMatplotDataDecomposer::fillVertices(char* id, float* buffer, int bu
 void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask,
     double* x, double* y, double* z, int numX, int numY)
 {
-    double xi;
-    double yj;
-    double zij;
-    double yjp1;
-    double xip1;
+    double xi = 0.;
+    double yj = 0.;
+    double zij = 0.;
+    double yjp1 = 0.;
+    double xip1 = 0.;
 
-    int i;
-    int j;
-
-    int bufferOffset;
+    int bufferOffset = 0;
 
 #if PER_VERTEX_VALUES
-    for (j = 0; j < numY; j++)
+    for (int j = 0; j < numY; j++)
     {
         yj = (double) j * y[1] + y[0];
 
@@ -94,7 +91,7 @@ void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLe
             }
         }
 
-        for (i = 0; i < numX; i++)
+        for (int i = 0; i < numX; i++)
         {
             xi = (double) i * x[1] + x[0];
             bufferOffset = elementsSize*(numX*j + i);
@@ -130,7 +127,7 @@ void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLe
 #else
     bufferOffset = 0;
 
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
         double ycoords[4];
         int yindices[4];
@@ -157,11 +154,10 @@ void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLe
         yindices[2] = j+1;
         yindices[3] = j+1;
 
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             double xcoords[4];
             int xindices[4];
-            int k;
 
             xi = (double) i * x[1] + x[0];
             xip1 = (double) (i+1) * x[1] + x[0];
@@ -182,7 +178,7 @@ void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLe
             xindices[2] = i;
             xindices[3] = i+1;
 
-            for (k = 0; k < 4; k++)
+            for (int k = 0; k < 4; k++)
             {
                 if (coordinateMask & 0x1)
                 {
@@ -221,12 +217,12 @@ void NgonGridMatplotDataDecomposer::fillGridVertices(float* buffer, int bufferLe
  */
 void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int bufferLength, int elementsSize)
 {
-    char* parent;
-    char* parentFigure;
+    char* parent = NULL;
+    char* parentFigure = NULL;
 
-    double* z;
-    double* colormap;
-    double currentZ;
+    double* z = NULL;
+    double* colormap = NULL;
+    double currentZ = 0.;
 
     int numX = 0;
     int* piNumX = &numX;
@@ -235,8 +231,6 @@ void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int buff
     int colormapSize = 0;
     int* piColormapSize = &colormapSize;
 
-    int i;
-    int j;
     int bufferOffset = 0;
 
     getGraphicObjectProperty(id, __GO_PARENT__, jni_string, (void**) &parent);
@@ -264,9 +258,9 @@ void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int buff
     }
 
 #if PER_VERTEX_VALUES
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             currentZ = z[i*(numY-1)+(numY-2-j)];
 
@@ -294,7 +288,7 @@ void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int buff
     }
 
     /* Last line */
-    for (i = 0; i < numX-1; i++)
+    for (int i = 0; i < numX-1; i++)
     {
         currentZ = z[(numY-1)*i+0];
 
@@ -320,9 +314,9 @@ void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int buff
 
 #else
 
-    for (j = 0; j < numY-1; j++)
+    for (int j = 0; j < numY-1; j++)
     {
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
             float facetColor[3];
 
@@ -345,9 +339,9 @@ void NgonGridMatplotDataDecomposer::fillColors(char* id, float* buffer, int buff
  */
 int NgonGridMatplotDataDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
     double zShift = 0.0;
     double* pdZShift = &zShift;
 
@@ -356,7 +350,7 @@ int NgonGridMatplotDataDecomposer::fillIndices(char* id, int* buffer, int buffer
     int numY = 0;
     int* piNumY = &numY;
 
-    int numberIndices;
+    int numberIndices = 0;
 
     NgonGridMatplotDataDecomposer* decomposer = get();
 
@@ -382,8 +376,8 @@ int NgonGridMatplotDataDecomposer::fillIndices(char* id, int* buffer, int buffer
 
 int NgonGridMatplotDataDecomposer::isFacetValid(double* z, double* values, int numX, int numY, int i, int j, int logUsed, int currentEdgeValid, int* nextEdgeValid)
 {
-    double zij;
-    int facetValid;
+    double zij = 0.;
+    int facetValid = 0;
 
     zij = getZCoordinate(z, numX, numY, i, j, logUsed);
 
index ec46b89..6524075 100644 (file)
@@ -344,11 +344,9 @@ int NgonPolylineData::setZCoordinatesShift(double const* data, int numElements)
 
 int NgonPolylineData::setNumElementsArray(int const* numElementsArray)
 {
-    int newNumElements;
-    int previousNumElements;
-    int result;
-
-    result = 1;
+    int newNumElements = 0;
+    int previousNumElements = 0;
+    int result = 1;
 
     if (numElementsArray[0] != 1)
     {
@@ -499,8 +497,7 @@ int NgonPolylineData::setNumElementsArray(int const* numElementsArray)
 
 void NgonPolylineData::copyShiftCoordinatesArray(double * newShift, double const* oldShift, int numElementsNew)
 {
-    int i;
-    int numElementsCopied;
+    int numElementsCopied = 0;
 
     if (numElementsNew < numVerticesPerGon)
     {
@@ -511,12 +508,12 @@ void NgonPolylineData::copyShiftCoordinatesArray(double * newShift, double const
         numElementsCopied = numVerticesPerGon;
     }
 
-    for (i = 0; i < numElementsCopied; i++)
+    for (int i = 0; i < numElementsCopied; i++)
     {
         newShift[i] = oldShift[i];
     }
 
-    for (i = numElementsCopied; i < numElementsNew; i++)
+    for (int i = numElementsCopied; i < numElementsNew; i++)
     {
         newShift[i] = 0.0;
     }
index 0bda6dc..e9a9b2b 100644 (file)
@@ -26,9 +26,9 @@ Plot3DDecomposer* Plot3DDecomposer::decomposer = NULL;
 
 void Plot3DDecomposer::fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
     int numX = 0;
     int* piNumX = &numX;
@@ -49,7 +49,7 @@ void Plot3DDecomposer::fillVertices(char* id, float* buffer, int bufferLength, i
 
 double Plot3DDecomposer::getZCoordinate(double* z, int numX, int numY, int i, int j)
 {
-    double zij;
+    double zij = 0.;
 
     zij = z[numX*j+i];
 
@@ -58,7 +58,7 @@ double Plot3DDecomposer::getZCoordinate(double* z, int numX, int numY, int i, in
 
 double Plot3DDecomposer::getZCoordinate(double* z, int numX, int numY, int i, int j, int logUsed)
 {
-    double zij;
+    double zij = 0.;
 
     zij = z[numX*j+i];
 
@@ -88,11 +88,11 @@ void Plot3DDecomposer::getFacetTriangles(double* x, double* y, double* z, int nu
  */
 void Plot3DDecomposer::fillColors(char* id, float* buffer, int bufferLength, int elementsSize)
 {
-    char* parentFigure;
-    char* parent;
+    char* parentFigure = NULL;
+    char* parent = NULL;
 
-    double* z;
-    double* colormap;
+    double* z = NULL;
+    double* colormap = NULL;
 
 
     int numX = 0;
@@ -133,16 +133,16 @@ void Plot3DDecomposer::fillColors(char* id, float* buffer, int bufferLength, int
 
 int Plot3DDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
     int numX = 0;
     int* piNumX = &numX;
     int numY = 0;
     int* piNumY = &numY;
 
-    int numberIndices;
+    int numberIndices = 0;
 
     Plot3DDecomposer* decomposer = get();
 
@@ -189,47 +189,43 @@ int Plot3DDecomposer::getWireIndicesSize(char* id)
  */
 int Plot3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* x;
-    double* y;
-    double* z;
+    double* x = NULL;
+    double* y = NULL;
+    double* z = NULL;
 
     int numX = 0;
     int* piNumX = &numX;
     int numY = 0;
     int* piNumY = &numY;
 
-    /* x-axis and y-axis indices (respectively) */
-    int i;
-    int j;
-
-    int previousRowValid;
-    int currentRowValid;
-    int nextRowValid;
-
-    int previousColumnValid;
-    int currentColumnValid;
-    int nextColumnValid;
-
-    int lowerLeftZValid;
-    int lowerRightZValid;
-    int upperLeftZValid;
-    int upperRightZValid;
-
-    int jm1HorizontalEdgeZValid;
-    int im1VerticalEdgeZValid;
-    int jHorizontalEdgeZValid;
-    int iVerticalEdgeZValid;
-    int jp1HorizontalEdgeZValid;
-    int ip1VerticalEdgeZValid;
-
-    int ij;
-    int ip1j;
-    int ijp1;
-    int ip1jp1;
-    int ijm1;
-    int ip1jm1;
-
-    int firstVertexIndex;
+    int previousRowValid = 0;
+    int currentRowValid = 0;
+    int nextRowValid = 0;
+
+    int previousColumnValid = 0;
+    int currentColumnValid = 0;
+    int nextColumnValid = 0;
+
+    int lowerLeftZValid = 0;
+    int lowerRightZValid = 0;
+    int upperLeftZValid = 0;
+    int upperRightZValid = 0;
+
+    int jm1HorizontalEdgeZValid = 0;
+    int im1VerticalEdgeZValid = 0;
+    int jHorizontalEdgeZValid = 0;
+    int iVerticalEdgeZValid = 0;
+    int jp1HorizontalEdgeZValid = 0;
+    int ip1VerticalEdgeZValid = 0;
+
+    int ij = 0;
+    int ip1j = 0;
+    int ijp1 = 0;
+    int ip1jp1 = 0;
+    int ijm1 = 0;
+    int ip1jm1 = 0;
+
+    int firstVertexIndex = 0;
 
     int bufferOffset = 0;
 
@@ -258,7 +254,8 @@ int Plot3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, i
     /* Set to 0 as it is not relevant for the first row iteration */
     jm1HorizontalEdgeZValid = 0;
 
-    for (j = 0; j < numY-1; j++)
+    /* x-axis and y-axis indices (respectively) */
+    for (int j = 0; j < numY-1; j++)
     {
         nextRowValid = DecompositionUtils::isValid(y[j+1]);
 
@@ -299,7 +296,7 @@ int Plot3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, i
         /* Set to 0 as not relevant for the first column iteration */
         im1VerticalEdgeZValid = 0;
 
-        for (i = 0; i < numX-1; i++)
+        for (int i = 0; i < numX-1; i++)
         {
 #if !PER_VERTEX_VALUES
             firstVertexIndex = getFirstVertexIndex(numX, numY, i, j);
@@ -430,7 +427,7 @@ int Plot3DDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, i
 
     ij = getPointIndex(numX, numY, 0, numY-1);
 
-    for (i = 0; i < numX-1; i++)
+    for (int i = 0; i < numX-1; i++)
     {
         nextColumnValid = DecompositionUtils::isValid(x[i+1]);
 
index b794b98..157eca4 100644 (file)
@@ -97,10 +97,10 @@ int PolylineDecomposer::getDataSize(char* id)
 
 void PolylineDecomposer::fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double* t;
-    double* xshift;
-    double* yshift;
-    double* zshift;
+    double* t = NULL;
+    double* xshift = NULL;
+    double* yshift = NULL;
+    double* zshift = NULL;
 
     int polylineStyle = 0;
     int* piPolylineStyle = &polylineStyle;
@@ -149,15 +149,12 @@ void PolylineDecomposer::fillSegmentsDecompositionVertices(char* id, float* buff
 {
 
     int componentIndices[3];
-    int i;
 
     // TODO Optimize ? (test if s = 1 and t = 0, coordinateMask = 0 ...)
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         /* Offset of a polyline vertex */
-        int v0;
-
-        v0 = elementsSize*i;
+        int v0 = elementsSize*i;
 
         componentIndices[0] = i;
         componentIndices[1] = i;
@@ -172,7 +169,7 @@ void PolylineDecomposer::fillSegmentsDecompositionVertices(char* id, float* buff
 void PolylineDecomposer::getAndWriteVertexToBuffer(float* buffer, int offset, double* coordinates, int* vertexIndices, int nPoints, int elementsSize,
     double* xshift, double* yshift, double* zshift, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double coordinate;
+    double coordinate = 0.;
 
     if (coordinateMask & 0x01)
     {
@@ -235,14 +232,12 @@ void PolylineDecomposer::getAndWriteVertexToBuffer(float* buffer, int offset, do
 void PolylineDecomposer::fillStairDecompositionVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation,
     int logMask, double* coordinates, int nPoints, double* xshift, double* yshift, double* zshift)
 {
-    int i;
-
     int closed = 0;
     int* piClosed = &closed;
 
     /* Offsets of the left and right vertices (respectively) */
-    int v0;
-    int v1;
+    int v0 = 0;
+    int v1 = 0;
 
     int componentIndices[3];
 
@@ -253,7 +248,7 @@ void PolylineDecomposer::fillStairDecompositionVertices(char* id, float* buffer,
 
     getGraphicObjectProperty(id, __GO_CLOSED__, jni_bool, (void**) &piClosed);
 
-    for (i = 0; i < nPoints-1; i++)
+    for (int i = 0; i < nPoints-1; i++)
     {
         v0 = elementsSize*2*i;
         v1 = elementsSize*(2*i+1);
@@ -302,17 +297,15 @@ void PolylineDecomposer::fillStairDecompositionVertices(char* id, float* buffer,
 void PolylineDecomposer::fillVerticalLinesDecompositionVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation,
     int logMask, double* coordinates, int nPoints, double* xshift, double* yshift, double* zshift)
 {
-    int i;
-
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         /* Offsets of the lower and upper vertices (respectively) */
-        int v0;
-        int v1;
+        int v0 = 0;
+        int v1 = 0;
 
         /* Coordinates of the lower and upper vertices (respectively) */
-        double coord0;
-        double coord1;
+        double coord0 = 0.;
+        double coord1 = 0.;
 
         v0 = elementsSize*2*i;
         v1 = elementsSize*(2*i+1);
@@ -433,10 +426,8 @@ void PolylineDecomposer::fillVerticalBarsDecompositionVertices(char* id, float*
     double barWidth = 0.0;
     double* pdBarWidth = &barWidth;
 
-    int i;
-
     int shiftUsed[3];
-    int *piShiftUsed;
+    int *piShiftUsed = NULL;
 
     /*
      * Offsets of the lower-left, lower-right, upper-right and upper-left bar vertices (respectively)
@@ -463,7 +454,7 @@ void PolylineDecomposer::fillVerticalBarsDecompositionVertices(char* id, float*
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Z_COORDINATES_SHIFT_SET__, jni_double_vector, (void**) &piShiftUsed);
 
 
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         offsets[0] = elementsSize*4*i;
         offsets[1] = elementsSize*(4*i+1);
@@ -557,10 +548,8 @@ void PolylineDecomposer::fillHorizontalBarsDecompositionVertices(char* id, float
     double barWidth = 0.0;
     double* pdBarWidth = &barWidth;
 
-    int i;
-
     int shiftUsed[3];
-    int *piShiftUsed;
+    int *piShiftUsed = NULL;
 
     /*
      * Offsets of the lower-left, lower-right, upper-right and upper-left bar vertices (respectively)
@@ -586,7 +575,7 @@ void PolylineDecomposer::fillHorizontalBarsDecompositionVertices(char* id, float
     piShiftUsed = &shiftUsed[2];
     getGraphicObjectProperty(id, __GO_DATA_MODEL_Z_COORDINATES_SHIFT_SET__, jni_double_vector, (void**) &piShiftUsed);
 
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         offsets[0] = elementsSize*4*i;
         offsets[1] = elementsSize*(4*i+1);
@@ -679,8 +668,8 @@ void PolylineDecomposer::fillHorizontalBarsDecompositionVertices(char* id, float
  */
 void PolylineDecomposer::fillColors(char* id, float* buffer, int bufferLength, int elementsSize)
 {
-    char* parent;
-    char* parentFigure;
+    char* parent = NULL;
+    char* parentFigure = NULL;
 
     int interpColorMode = 0;
     int* piInterpColorMode = &interpColorMode;
@@ -691,9 +680,9 @@ void PolylineDecomposer::fillColors(char* id, float* buffer, int bufferLength, i
     int colormapSize = 0;
     int* piColormapSize = &colormapSize;
     int bufferOffset = 0;
-    int* interpColorVector;
+    int* interpColorVector = NULL;
 
-    double* colormap;
+    double* colormap = NULL;
 
     getGraphicObjectProperty(id, __GO_INTERP_COLOR_MODE__, jni_bool, (void**) &piInterpColorMode);
 
@@ -770,8 +759,8 @@ void PolylineDecomposer::fillColors(char* id, float* buffer, int bufferLength, i
 
 void PolylineDecomposer::fillTextureCoordinates(char* id, float* buffer, int bufferLength)
 {
-    char* parent;
-    char* parentFigure;
+    char* parent = NULL;
+    char* parentFigure = NULL;
 
     int interpColorMode = 0;
     int* piInterpColorMode = &interpColorMode;
@@ -782,9 +771,9 @@ void PolylineDecomposer::fillTextureCoordinates(char* id, float* buffer, int buf
     int colormapSize = 0;
     int* piColormapSize = &colormapSize;
     int bufferOffset = 0;
-    int* interpColorVector;
+    int* interpColorVector = NULL;
 
-    double* colormap;
+    double* colormap = NULL;
 
     getGraphicObjectProperty(id, __GO_INTERP_COLOR_MODE__, jni_bool, (void**) &piInterpColorMode);
 
@@ -916,7 +905,7 @@ int PolylineDecomposer::getIndicesSize(char* id)
 
 int PolylineDecomposer::getArrowTriangleIndicesSize(int nPoints, int closed)
 {
-    int nIndices;
+    int nIndices = 0;
 
     if (nPoints < 2)
     {
@@ -948,10 +937,10 @@ int PolylineDecomposer::getBarsDecompositionTriangleIndicesSize(int nPoints)
  */
 int PolylineDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* coordinates;
-    double* xshift;
-    double* yshift;
-    double* zshift;
+    double* coordinates = NULL;
+    double* xshift = NULL;
+    double* yshift = NULL;
+    double* zshift = NULL;
 
     int nPoints = 0;
     int* piNPoints = &nPoints;
@@ -1190,10 +1179,10 @@ int PolylineDecomposer::fillArrowTriangleIndices(char* id, int* buffer, int buff
     int closed = 0;
     int* piClosed = &closed;
 
-    int currentValid;
-    int nextValid;
+    int currentValid = 0;
+    int nextValid = 0;
 
-    int firstArrowVertex;
+    int firstArrowVertex = 0;
     int nArrows = 0;
 
     int offset = 0;
@@ -1272,7 +1261,6 @@ int PolylineDecomposer::fillBarsDecompositionTriangleIndices(char* id, int* buff
     double coordsi[3];
 
     int triangleIndices[6];
-    int i;
     int offset = 0;
     int numberValidIndices = 0;
 
@@ -1296,7 +1284,7 @@ int PolylineDecomposer::fillBarsDecompositionTriangleIndices(char* id, int* buff
     DecompositionUtils::getDecomposedRectangleTriangleIndices(triangleIndices);
 
     /* Bars */
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i, &coordsi[0], &coordsi[1], &coordsi[2]);
 
@@ -1468,10 +1456,10 @@ int PolylineDecomposer::getBarsDecompositionSegmentIndicesSize(int nPoints, int
 
 int PolylineDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* coordinates;
-    double* xshift;
-    double* yshift;
-    double* zshift;
+    double* coordinates = NULL;
+    double* xshift = NULL;
+    double* yshift = NULL;
+    double* zshift = NULL;
 
     int polylineStyle = 0;
     int* piPolylineStyle = &polylineStyle;
@@ -1526,10 +1514,8 @@ int PolylineDecomposer::fillSegmentsDecompositionSegmentIndices(char* id, int* b
 {
     double coordsi[3];
 
-    int i;
-
-    int currentValid;
-    int nextValid;
+    int currentValid = 0;
+    int nextValid = 0;
 
     int offset = 0;
     int numberValidIndices = 0;
@@ -1554,7 +1540,7 @@ int PolylineDecomposer::fillSegmentsDecompositionSegmentIndices(char* id, int* b
         currentValid &= DecompositionUtils::isLogValid(coordsi[0], coordsi[1], coordsi[2], logMask);
     }
 
-    for (i = 0; i < nPoints-1; i++)
+    for (int i = 0; i < nPoints-1; i++)
     {
         getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i+1, &coordsi[0], &coordsi[1], &coordsi[2]);
 
@@ -1606,11 +1592,9 @@ int PolylineDecomposer::fillStairDecompositionSegmentIndices(char* id, int* buff
     double coordsi[3];
     double coordsip1[3];
 
-    int i;
-
-    int currentValid;
-    int middleVertexValid;
-    int nextValid;
+    int currentValid = 0;
+    int middleVertexValid = 0;
+    int nextValid = 0;
 
     int offset = 0;
     int numberValidIndices = 0;
@@ -1635,7 +1619,7 @@ int PolylineDecomposer::fillStairDecompositionSegmentIndices(char* id, int* buff
         currentValid &= DecompositionUtils::isLogValid(coordsi[0], coordsi[1], coordsi[2], logMask);
     }
 
-    for (i = 0; i < nPoints-1; i++)
+    for (int i = 0; i < nPoints-1; i++)
     {
         getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i+1, &coordsip1[0], &coordsip1[1], &coordsip1[2]);
 
@@ -1712,7 +1696,6 @@ int PolylineDecomposer::fillVerticalLinesDecompositionSegmentIndices(char* id, i
 {
     double coordsi[3];
 
-    int i;
     int offset = 0;
     int numberValidIndices = 0;
 
@@ -1722,7 +1705,7 @@ int PolylineDecomposer::fillVerticalLinesDecompositionSegmentIndices(char* id, i
     }
 
     /* Vertical lines */
-    for (i = 0; i < nPoints; i++)
+    for (int i = 0; i < nPoints; i++)
     {
         getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i, &coordsi[0], &coordsi[1], &coordsi[2]);
 
@@ -1756,7 +1739,7 @@ int PolylineDecomposer::fillVerticalLinesDecompositionSegmentIndices(char* id, i
             currentValid &= DecompositionUtils::isLogValid(coordsi[0], coordsi[1], coordsi[2], logMask);
         }
 
-        for (i = 0; i < nPoints-1; i++)
+        for (int i = 0; i < nPoints-1; i++)
         {
             getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i+1, &coordsi[0], &coordsi[1], &coordsi[2]);
 
@@ -1791,8 +1774,7 @@ int PolylineDecomposer::fillBarsDecompositionSegmentIndices(char* id, int* buffe
     double* pdBarWidth = &barWidth;
     double coordsi[3];
 
-    int barWidthValid;
-    int i;
+    int barWidthValid = 0;
     int offset = 0;
     int numberValidIndices = 0;
 
@@ -1809,7 +1791,7 @@ int PolylineDecomposer::fillBarsDecompositionSegmentIndices(char* id, int* buffe
     if (barWidthValid)
     {
         /* Bars */
-        for (i = 0; i < nPoints; i++)
+        for (int i = 0; i < nPoints; i++)
         {
             getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i, &coordsi[0], &coordsi[1], &coordsi[2]);
 
@@ -1854,7 +1836,7 @@ int PolylineDecomposer::fillBarsDecompositionSegmentIndices(char* id, int* buffe
             currentValid &= DecompositionUtils::isLogValid(coordsi[0], coordsi[1], coordsi[2], logMask);
         }
 
-        for (i = 0; i < nPoints-1; i++)
+        for (int i = 0; i < nPoints-1; i++)
         {
             getShiftedPolylinePoint(coordinates, xshift, yshift, zshift, nPoints, i+1, &coordsi[0], &coordsi[1], &coordsi[2]);
 
index 8d49076..d84556c 100644 (file)
@@ -103,7 +103,7 @@ int ScilabView::getNbFigure(void)
 void ScilabView::createObject(char *pstId)
 {
     //std::cerr << "[ScilabView] ++ createObject UID=" << pstId << std::endl;
-    char *pstType;
+    char *pstType = NULL;
 
     getGraphicObjectProperty(pstId, __GO_TYPE__, jni_string, (void **)&pstType);
     if (pstType != NULL && strcmp(pstType, __GO_FIGURE__) == 0)
index 6202315..ba5da5c 100644 (file)
@@ -180,9 +180,7 @@ unsigned int TriangleMeshData::getNumVertices(void)
  */
 int TriangleMeshData::setNumVertices(unsigned int numVertices)
 {
-    int result;
-
-    result = 1;
+    int result = 1;
 
     if (numVertices == 0 && numberVertices > 0)
     {
@@ -260,9 +258,7 @@ unsigned int TriangleMeshData::getNumIndices(void)
 
 int TriangleMeshData::setNumIndices(unsigned int numIndices)
 {
-    int result;
-
-    result = 1;
+    int result = 1;
 
     if (numIndices != this->numberTriangles)
     {
index cf35e28..800fba0 100644 (file)
@@ -123,9 +123,7 @@ unsigned int TriangleMeshFecData::getNumIndices(void)
 
 int TriangleMeshFecData::setNumIndices(unsigned int numIndices)
 {
-    int result;
-
-    result = 1;
+    int result = 1;
 
     if (numIndices != this->numberTriangles)
     {
index 94deb58..150b6ef 100644 (file)
@@ -35,16 +35,15 @@ int TriangleMeshFecDataDecomposer::getDataSize(char* id)
 
 void TriangleMeshFecDataDecomposer::fillVertices(char* id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
 {
-    double* coordinates;
+    double* coordinates = NULL;
 
     int numVertices = 0;
     int* piNumVertices = &numVertices;
-    int i;
 
     getGraphicObjectProperty(id, __GO_DATA_MODEL_COORDINATES__, jni_double_vector, (void**) &coordinates);
     getGraphicObjectProperty(id, __GO_DATA_MODEL_NUM_VERTICES__, jni_int, (void**) &piNumVertices);
 
-    for (i = 0; i < numVertices; i++)
+    for (int i = 0; i < numVertices; i++)
     {
         buffer[elementsSize*i] = (float)coordinates[3*i];
         buffer[elementsSize*i +1] = (float)coordinates[3*i+1];
@@ -60,25 +59,24 @@ void TriangleMeshFecDataDecomposer::fillVertices(char* id, float* buffer, int bu
 
 void TriangleMeshFecDataDecomposer::fillTextureCoordinates(char* id, float* buffer, int bufferLength)
 {
-    char* parentFigure;
+    char* parentFigure = NULL;
     int colormapSize = 0;
     int* piColormapSize = &colormapSize;
-    int* colorRange;
+    int* colorRange = NULL;
 
-    double colorsNumber;
-    double scale;
-    double t;
+    double colorsNumber = 0.;
+    double scale = 0.;
+    double t = 0.;
 
-    double* values;
-    double* zBounds;
+    double* values = NULL;
+    double* zBounds = NULL;
 
-    double minValue;
-    double maxValue;
+    double minValue = 0.;
+    double maxValue = 0.;
 
     int numVertices = 0;
     int* piNumVertices = &numVertices;
 
-    int i;
     int bufferOffset = 0;
 
     getGraphicObjectProperty(id, __GO_PARENT_FIGURE__, jni_string, (void**) &parentFigure);
@@ -123,7 +121,7 @@ void TriangleMeshFecDataDecomposer::fillTextureCoordinates(char* id, float* buff
 
     if (maxValue == minValue)
     {
-        for (i = 0; i < numVertices; i++)
+        for (int i = 0; i < numVertices; i++)
         {
             buffer[bufferOffset++] = (float)minValue;
             buffer[bufferOffset++] = 0;
@@ -133,7 +131,7 @@ void TriangleMeshFecDataDecomposer::fillTextureCoordinates(char* id, float* buff
     }
     else
     {
-        for (i = 0; i < numVertices; i++)
+        for (int i = 0; i < numVertices; i++)
         {
             buffer[bufferOffset++] = (float)(t + scale * (values[i] - minValue) / (maxValue - minValue));
             buffer[bufferOffset++] = 0;
@@ -145,16 +143,16 @@ void TriangleMeshFecDataDecomposer::fillTextureCoordinates(char* id, float* buff
 
 void TriangleMeshFecDataDecomposer::fillColors(char* id, float* buffer, int bufferLength, int elementsSize)
 {
-    char* parent;
-    char* parentFigure;
+    char* parent = NULL;
+    char* parentFigure = NULL;
 
-    double* values;
-    double* zBounds;
-    double* colormap;
+    double* values = NULL;
+    double* zBounds = NULL;
+    double* colormap = NULL;
 
-    double minValue;
-    double maxValue;
-    double valueRange;
+    double minValue = 0.;
+    double maxValue = 0.;
+    double valueRange = 0.;
 
     float minColor[3];
     float maxColor[3];
@@ -165,12 +163,11 @@ void TriangleMeshFecDataDecomposer::fillColors(char* id, float* buffer, int buff
     int numVertices = 0;
     int* piNumVertices = &numVertices;
 
-    int minColorIndex;
-    int maxColorIndex;
+    int minColorIndex = 0;
+    int maxColorIndex = 0;
 
-    int* colorRange;
+    int* colorRange = NULL;
     int useOutsideColors = 0;
-    int i;
     int bufferOffset = 0;
 
     getGraphicObjectProperty(id, __GO_PARENT__, jni_string, (void**) &parent);
@@ -255,7 +252,7 @@ void TriangleMeshFecDataDecomposer::fillColors(char* id, float* buffer, int buff
         valueRange = maxValue - minValue;
     }
 
-    for (i = 0; i < numVertices; i++)
+    for (int i = 0; i < numVertices; i++)
     {
         bufferOffset = elementsSize*i;
 
@@ -295,19 +292,12 @@ void TriangleMeshFecDataDecomposer::fillColors(char* id, float* buffer, int buff
 
 void TriangleMeshFecDataDecomposer::computeMinMaxValues(double* values, int numValues, double* valueMin, double* valueMax)
 {
-    double maxDouble;
-    double tmpValueMin;
-    double tmpValueMax;
-    double value;
+    double maxDouble = DecompositionUtils::getMaxDoubleValue();
+    double tmpValueMin = maxDouble;
+    double tmpValueMax = -maxDouble;
+    double value = 0.;
 
-    int i;
-
-    maxDouble = DecompositionUtils::getMaxDoubleValue();
-
-    tmpValueMin = maxDouble;
-    tmpValueMax = -maxDouble;
-
-    for (i = 0; i < numValues; i++)
+    for (int i = 0; i < numValues; i++)
     {
         value = values[i];
 
@@ -338,21 +328,19 @@ int TriangleMeshFecDataDecomposer::getIndicesSize(char* id)
 
 int TriangleMeshFecDataDecomposer::fillIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* coordinates;
-    double* values;
+    double* coordinates = NULL;
+    double* values = NULL;
 
     int numIndices = 0;
     int* piNumIndices = &numIndices;
     int numVertices = 0;
     int* piNumVertices = &numVertices;
 
-    int* triangleIndices;
-
-    int i;
+    int* triangleIndices = NULL;
 
-    int v0;
-    int v1;
-    int v2;
+    int v0 = 0;
+    int v1 = 0;
+    int v2 = 0;
     int bufferOffset = 0;
 
     getGraphicObjectProperty(id, __GO_DATA_MODEL_NUM_INDICES__, jni_int, (void**) &piNumIndices);
@@ -369,7 +357,7 @@ int TriangleMeshFecDataDecomposer::fillIndices(char* id, int* buffer, int buffer
 
     getGraphicObjectProperty(id, __GO_DATA_MODEL_INDICES__, jni_int_vector, (void**) &triangleIndices);
 
-    for (i = 0; i < numIndices; i++)
+    for (int i = 0; i < numIndices; i++)
     {
         v0 = triangleIndices[3*i];
         v1 = triangleIndices[3*i+1];
@@ -456,19 +444,18 @@ int TriangleMeshFecDataDecomposer::getWireIndicesSize(char* id)
  */
 int TriangleMeshFecDataDecomposer::fillWireIndices(char* id, int* buffer, int bufferLength, int logMask)
 {
-    double* coordinates;
-    double* values;
+    double* coordinates = NULL;
+    double* values = NULL;
 
     int numVertices = 0;
     int* piNumVertices = &numVertices;
     int numIndices = 0;
     int* piNumIndices = &numIndices;
-    int* triangleIndices;
+    int* triangleIndices = NULL;
 
-    int i;
-    int v0;
-    int v1;
-    int v2;
+    int v0 = 0;
+    int v1 = 0;
+    int v2 = 0;
     int bufferOffset = 0;
 
     getGraphicObjectProperty(id, __GO_DATA_MODEL_NUM_INDICES__, jni_int, (void**) &piNumIndices);
@@ -485,7 +472,7 @@ int TriangleMeshFecDataDecomposer::fillWireIndices(char* id, int* buffer, int bu
 
     getGraphicObjectProperty(id, __GO_DATA_MODEL_INDICES__, jni_int_vector, (void**) &triangleIndices);
 
-    for (i = 0; i < numIndices; i++)
+    for (int i = 0; i < numIndices; i++)
     {
         v0 = triangleIndices[3*i];
         v1 = triangleIndices[3*i+1];
index 7c92699..1278b39 100644 (file)
 
 void Triangulator::determineSmallestAxis(void)
 {
-    double minval;
+    double minval = 0.;
 
-    Vector3d min;
-    Vector3d max;
-
-    min = inputPoints[0];
-    max = min;
+    Vector3d min = inputPoints[0];
+    Vector3d max = min;
 
     for (int i = 1 ; i < numPoints; i++)
     {
@@ -92,11 +89,10 @@ void Triangulator::fillPoints(void)
 double Triangulator::computeArea(void)
 {
     double area = 0.0;
-    int ip1;
 
     for (int i = 0; i < numPoints; i++)
     {
-        ip1 = (i+1) % numPoints;
+        int ip1 = (i+1) % numPoints;
 
         area += ((points[i].x*points[ip1].y)-(points[i].y*points[ip1].x));
     }
@@ -126,7 +122,7 @@ void Triangulator::fillVertexIndices(void)
 
 void Triangulator::fillConvexVerticesList(void)
 {
-    double dp;
+    double dp = 0.;
     std::list<int>::iterator vi, vim1, vip1;
 
     flagList.resize(vertexIndices.size());
@@ -153,7 +149,7 @@ void Triangulator::fillConvexVerticesList(void)
 void Triangulator::fillEarList(void)
 {
     std::list<int>::iterator vi;
-    int res;
+    int res = 0;
 
     for (vi = vertexIndices.begin(); vi != vertexIndices.end(); vi++)
     {
@@ -243,8 +239,8 @@ int Triangulator::isAnEar(std::list<int>::iterator vertex)
 /* To do: streamline */
 void Triangulator::updateVertex(std::list<int>::iterator vertex)
 {
-    double dp;
-    int res;
+    double dp = 0.;
+    int res = 0;
 
     std::list<int>::iterator pred, succ;
 
@@ -312,17 +308,11 @@ void Triangulator::updateVertex(std::list<int>::iterator vertex)
 
 double Triangulator::computeDotProduct(int im1, int i, int ip1)
 {
-    double dp;
-    Vector3d eim1;
-    Vector3d ei;
-    Vector3d eim1p;
-
-    eim1 = minus(points[i], points[im1]);
-    ei = minus(points[ip1], points[i]);
-
+    double dp = 0.;
+    Vector3d eim1 = minus(points[i], points[im1]);
+    Vector3d ei = minus(points[ip1], points[i]);
+    Vector3d eim1p = perpendicularVector(eim1);
     /* Ought to use cross product */
-    eim1p = perpendicularVector(eim1);
-
     dp = dot(eim1p, ei);
 
     return dp;
@@ -330,9 +320,9 @@ double Triangulator::computeDotProduct(int im1, int i, int ip1)
 
 int Triangulator::pointInTriangle(Vector3d A, Vector3d B, Vector3d C, Vector3d P)
 {
-    double dot00, dot01, dot02, dot11, dot12;
-    double invDenom;
-    double u, v;
+    double dot00 = 0., dot01 = 0., dot02 = 0., dot11 = 0., dot12 = 0.;
+    double invDenom = 0.;
+    double u = 0., v = 0.;
 
     Vector3d v0, v1, v2;
 
@@ -395,7 +385,7 @@ Triangulator::Triangulator(void)
 
 void Triangulator::initialize(void)
 {
-    double area;
+    double area = 0.;
 
     numPoints = (int)inputPoints.size();
 
@@ -430,7 +420,7 @@ void Triangulator::addPoint(double x, double y, double z)
 
 void Triangulator::triangulate(void)
 {
-    int triIndex;
+    int triIndex = 0;
     std::list<int>::iterator it;
     std::list<int>::iterator vertex, pred, succ;
 
index 644bd55..4fb8950 100644 (file)
@@ -22,9 +22,9 @@ DataModel *DataModel::m_me = NULL;
 
 BOOL DataModel::setGraphicObjectProperty(char const* _pstID, char const* _pstName, void const* _dblValue, int numElements)
 {
-    Data3D* dataObject;
-    int property;
-    int returnValue;
+    Data3D* dataObject = NULL;
+    int property = 0;
+    int returnValue = 0;
 
     dataObject = (*m_dataMap)[std::string(_pstID)];
 
@@ -37,8 +37,8 @@ BOOL DataModel::setGraphicObjectProperty(char const* _pstID, char const* _pstNam
 
 void DataModel::getGraphicObjectProperty(char const* _pstID, char const* _pstName, void **_pvData)
 {
-    Data3D* dataObject;
-    int property;
+    Data3D* dataObject = NULL;
+    int property = 0;
 
     dataObject = (*m_dataMap)[std::string(_pstID)];
 
@@ -49,8 +49,8 @@ void DataModel::getGraphicObjectProperty(char const* _pstID, char const* _pstNam
 
 void DataModel::getGraphicObjectIntProperty(char const* _pstID, char const* _pstName, void **_pvData)
 {
-    Data3D* dataObject;
-    int property;
+    Data3D* dataObject = NULL;
+    int property = 0;
 
     property = (*m_dataMap)[std::string(_pstID)]->getPropertyFromName(_pstName);
 
@@ -63,7 +63,7 @@ void DataModel::getGraphicObjectIntProperty(char const* _pstID, char const* _pst
 
 char const* DataModel::createDataObject(char const* _pstID, char const* _sType)
 {
-    Data3D* newObject;
+    Data3D* newObject = NULL;
 
     /*
      * To be implemented as the Java graphicObject class' similar function
index ec0b1b1..f10c679 100644 (file)
@@ -34,9 +34,9 @@ BOOL setGraphicObjectProperty(char const* _pstID, char const* _pstName, void *_p
 {
     bool result = false;
 
-    double doubleValue;
-    int intValue;
-    BOOL boolValue;
+    double doubleValue = 0.;
+    int intValue = 0;
+    BOOL boolValue = FALSE;
 
     // Special Case for data, no need to go through Java.
     if (strncmp(_pstName, __GO_DATA_MODEL__, strlen(__GO_DATA_MODEL__)) == 0)