Remove useless asDouble argument from fscanfMat 02/10502/2
Sylvestre Ledru [Sat, 16 Feb 2013 10:52:29 +0000 (11:52 +0100)]
Change-Id: Ie9c076aa789f176e9583d41baadf1fcafe92f767

scilab/modules/fileio/sci_gateway/c/sci_fscanfMat.c
scilab/modules/fileio/src/c/fscanfMat.c
scilab/modules/fileio/src/c/fscanfMat.h

index 8b047a8..0cf3794 100644 (file)
@@ -2,11 +2,11 @@
  * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
  * Copyright (C) 2006 - INRIA - Allan CORNET
  * Copyright (C) 2010 - 2011 - DIGITEO - Allan CORNET
- * 
+ *
  * This file must be used under the terms of the CeCILL.
  * This source file is licensed as described in the file COPYING, which
  * you should have received as part of this distribution.  The terms
- * are also available at    
+ * are also available at
  * http://www.cecill.info/licences/Licence_CeCILL_V2-en.txt
  *
  */
@@ -25,7 +25,7 @@
 #endif
 #include "fscanfMat.h"
 /*--------------------------------------------------------------------------*/
-int sci_fscanfMat(char *fname,unsigned long fname_len)
+int sci_fscanfMat(char *fname, unsigned long fname_len)
 {
     SciErr sciErr;
     int *piAddressVarOne = NULL;
@@ -41,7 +41,7 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
     fscanfMatResult *results = NULL;
 
     Nbvars = 0;
-    CheckRhs(1, 3); 
+    CheckRhs(1, 3);
     CheckLhs(1, 2);
 
     if (Rhs == 3)
@@ -51,7 +51,7 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
         int iType3 = 0;
 
         sciErr = getVarAddressFromPosition(pvApiCtx, 3, &piAddressVarThree);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 3);
@@ -59,7 +59,7 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
         }
 
         sciErr = getVarType(pvApiCtx, piAddressVarThree, &iType3);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 3);
@@ -68,27 +68,27 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
 
         if (iType3 != sci_strings)
         {
-            Scierror(999,_("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 3);
+            Scierror(999, _("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 3);
             return 0;
         }
 
         sciErr = getVarDimension(pvApiCtx, piAddressVarThree, &m3, &n3);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 3);
             return 0;
         }
 
-        if ( (m3 != n3) && (n3 != 1) ) 
+        if ( (m3 != n3) && (n3 != 1) )
         {
-            Scierror(999,_("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 3);
+            Scierror(999, _("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 3);
             return 0;
         }
 
         if (getAllocatedSingleString(pvApiCtx, piAddressVarThree, &separator))
         {
-            Scierror(999,_("%s: Memory allocation error.\n"), fname);
+            Scierror(999, _("%s: Memory allocation error.\n"), fname);
             return 0;
         }
         bIsDefaultSeparator = FALSE;
@@ -105,18 +105,26 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
         int iType2 = 0;
 
         sciErr = getVarAddressFromPosition(pvApiCtx, 2, &piAddressVarTwo);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
-            if (separator) {FREE(separator); separator = NULL;}
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 2);
             return 0;
         }
 
         sciErr = getVarType(pvApiCtx, piAddressVarTwo, &iType2);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
-            if (separator) {FREE(separator); separator = NULL;}
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 2);
             return 0;
@@ -124,31 +132,47 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
 
         if (iType2 != sci_strings)
         {
-            if (separator) {FREE(separator); separator = NULL;}
-            Scierror(999,_("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 2);
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
+            Scierror(999, _("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 2);
             return 0;
         }
 
         sciErr = getVarDimension(pvApiCtx, piAddressVarTwo, &m2, &n2);
-        if(sciErr.iErr)
+        if (sciErr.iErr)
         {
-            if (separator) {FREE(separator); separator = NULL;}
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
             printError(&sciErr, 0);
             Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 2);
             return 0;
         }
 
-        if ( (m2 != n2) && (n2 != 1) ) 
+        if ( (m2 != n2) && (n2 != 1) )
         {
-            if (separator) {FREE(separator); separator = NULL;}
-            Scierror(999,_("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 2);
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
+            Scierror(999, _("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 2);
             return 0;
         }
 
         if (getAllocatedSingleString(pvApiCtx, piAddressVarTwo, &Format))
         {
-            if (separator) {FREE(separator); separator = NULL;}
-            Scierror(999,_("%s: Memory allocation error.\n"), fname);
+            if (separator)
+            {
+                FREE(separator);
+                separator = NULL;
+            }
+            Scierror(999, _("%s: Memory allocation error.\n"), fname);
             return 0;
         }
     }
@@ -158,20 +182,36 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
     }
 
     sciErr = getVarAddressFromPosition(pvApiCtx, 1, &piAddressVarOne);
-    if(sciErr.iErr)
+    if (sciErr.iErr)
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
         printError(&sciErr, 0);
         Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 1);
         return 0;
     }
 
     sciErr = getVarType(pvApiCtx, piAddressVarOne, &iType1);
-    if(sciErr.iErr)
+    if (sciErr.iErr)
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
         printError(&sciErr, 0);
         Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 1);
         return 0;
@@ -179,42 +219,74 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
 
     if (iType1 != sci_strings)
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
-        Scierror(999,_("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 1);
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
+        Scierror(999, _("%s: Wrong type for input argument #%d: A string expected.\n"), fname, 1);
         return 0;
     }
 
     sciErr = getVarDimension(pvApiCtx, piAddressVarOne, &m1, &n1);
-    if(sciErr.iErr)
+    if (sciErr.iErr)
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
         printError(&sciErr, 0);
         Scierror(999, _("%s: Can not read input argument #%d.\n"), fname, 1);
         return 0;
     }
 
-    if ( (m1 != n1) && (n1 != 1) ) 
+    if ( (m1 != n1) && (n1 != 1) )
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
-        Scierror(999,_("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 1);
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
+        Scierror(999, _("%s: Wrong size for input argument #%d: A string expected.\n"), fname, 1);
         return 0;
     }
 
     if (getAllocatedSingleString(pvApiCtx, piAddressVarOne, &filename))
     {
-        if (separator) {FREE(separator); separator = NULL;}
-        if (Format) {FREE(Format); Format = NULL;}
-        Scierror(999,_("%s: Memory allocation error.\n"), fname);
+        if (separator)
+        {
+            FREE(separator);
+            separator = NULL;
+        }
+        if (Format)
+        {
+            FREE(Format);
+            Format = NULL;
+        }
+        Scierror(999, _("%s: Memory allocation error.\n"), fname);
         return 0;
     }
 
     expandedFilename = expandPathVariable(filename);
     if (bIsDefaultSeparator)
     {
-        #define NB_DEFAULT_SUPPORTED_SEPARATORS 2
+#define NB_DEFAULT_SUPPORTED_SEPARATORS 2
 
         /* bug 8148 */
         /* default separator can be a space or a tabulation */
@@ -223,8 +295,8 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
 
         for (i = 0; i < NB_DEFAULT_SUPPORTED_SEPARATORS; i++)
         {
-            results = fscanfMat(expandedFilename, Format, supportedSeparators[i], TRUE);
-            if (results && results->err == FSCANFMAT_NO_ERROR)  
+            results = fscanfMat(expandedFilename, Format, supportedSeparators[i]);
+            if (results && results->err == FSCANFMAT_NO_ERROR)
             {
                 break;
             }
@@ -232,23 +304,35 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
     }
     else
     {
-        results = fscanfMat(expandedFilename, Format, separator, TRUE);
+        results = fscanfMat(expandedFilename, Format, separator);
     }
 
-    if (expandedFilename) {FREE(expandedFilename); expandedFilename = NULL;}
-    if (Format) {FREE(Format); Format = NULL;}
-    if (separator) {FREE(separator); separator = NULL;}
+    if (expandedFilename)
+    {
+        FREE(expandedFilename);
+        expandedFilename = NULL;
+    }
+    if (Format)
+    {
+        FREE(Format);
+        Format = NULL;
+    }
+    if (separator)
+    {
+        FREE(separator);
+        separator = NULL;
+    }
 
     if (results)
     {
-        switch(results->err)
+        switch (results->err)
         {
-        case FSCANFMAT_NO_ERROR:
+            case FSCANFMAT_NO_ERROR:
             {
                 if ( (results->values) && (results->m > 0) && (results->n > 0))
                 {
                     sciErr = createMatrixOfDouble(pvApiCtx, Rhs + 1, results->m, results->n, results->values);
-                    if(sciErr.iErr)
+                    if (sciErr.iErr)
                     {
                         freeFscanfMatResult(results);
                         results = NULL;
@@ -277,7 +361,7 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
                         freeFscanfMatResult(results);
                         results = NULL;
 
-                        if(sciErr.iErr)
+                        if (sciErr.iErr)
                         {
                             printError(&sciErr, 0);
                             return 0;
@@ -303,45 +387,53 @@ int sci_fscanfMat(char *fname,unsigned long fname_len)
                 freeFscanfMatResult(results);
                 results = NULL;
 
-                if (filename) {FREE(filename); filename = NULL;}
+                if (filename)
+                {
+                    FREE(filename);
+                    filename = NULL;
+                }
 
                 PutLhsVar();
             }
             break;
-        case FSCANFMAT_MOPEN_ERROR:
+            case FSCANFMAT_MOPEN_ERROR:
             {
-                Scierror(999,_("%s: can not open file %s.\n"), fname, filename);
+                Scierror(999, _("%s: can not open file %s.\n"), fname, filename);
             }
             break;
-        case FSCANFMAT_READLINES_ERROR:
+            case FSCANFMAT_READLINES_ERROR:
             {
-                Scierror(999,_("%s: can not read file %s.\n"), fname, filename);
+                Scierror(999, _("%s: can not read file %s.\n"), fname, filename);
             }
             break;
-        case FSCANFMAT_FORMAT_ERROR:
+            case FSCANFMAT_FORMAT_ERROR:
             {
-                Scierror(999,_("%s: Invalid format.\n"), fname);
+                Scierror(999, _("%s: Invalid format.\n"), fname);
             }
             break;
-        case FSCANFMAT_MEMORY_ALLOCATION:
+            case FSCANFMAT_MEMORY_ALLOCATION:
             {
-                Scierror(999,_("%s: Memory allocation error.\n"), fname);
+                Scierror(999, _("%s: Memory allocation error.\n"), fname);
             }
             break;
-        default:
-        case FSCANFMAT_ERROR:
+            default:
+            case FSCANFMAT_ERROR:
             {
-                Scierror(999,_("%s: error.\n"), fname);
+                Scierror(999, _("%s: error.\n"), fname);
             }
             break;
         }
     }
     else
     {
-        Scierror(999,_("%s: Memory allocation error.\n"), fname);
+        Scierror(999, _("%s: Memory allocation error.\n"), fname);
     }
 
-    if (filename) {FREE(filename); filename = NULL;}
+    if (filename)
+    {
+        FREE(filename);
+        filename = NULL;
+    }
 
     return 0;
 }
index 0199189..74f6833 100644 (file)
@@ -70,7 +70,7 @@ static char **removeEmptyLinesAtTheEnd(char **lines, int *sizelines);
 static BOOL isValidLineWithOnlyOneNumber(char *line);
 static char ** removeTextLinesAtTheEnd(char **lines, int *sizelines, char *format, char *separator);
 /*--------------------------------------------------------------------------*/
-fscanfMatResult *fscanfMat(char *filename, char *format, char *separator, BOOL asDouble)
+fscanfMatResult *fscanfMat(char *filename, char *format, char *separator)
 {
     int fd = 0;
     int f_swap = 0;
@@ -371,7 +371,10 @@ static int getNbColumnsInLine(char *line, char *format, char *separator)
                         {
                             freeArrayOfString(splittedStr, nbTokens);
                             /* bug 6889 */
-                            if (nbColums) nbColums--;
+                            if (nbColums)
+                            {
+                                nbColums--;
+                            }
                             FREE(str);
                             str = NULL;
                             return nbColums;
@@ -520,7 +523,10 @@ static double *getDoubleValuesFromLines(char **lines, int sizelines,
 {
     double *dValues = NULL;
 
-    if (m == 0 || n == 0) return NULL;
+    if (m == 0 || n == 0)
+    {
+        return NULL;
+    }
 
     dValues = (double*) MALLOC(sizeof(double) * (m * n));
     if (dValues)
@@ -627,7 +633,10 @@ static double returnINF(BOOL bPositive)
 {
     double v = 0;
     double p = 10;
-    if (!bPositive) p = -10;
+    if (!bPositive)
+    {
+        p = -10;
+    }
     return (double) p / (double)v;
 }
 /*--------------------------------------------------------------------------*/
@@ -766,7 +775,10 @@ static BOOL isOnlyBlankLine(const char *line)
         int i = 0;
         for (i = 0; i < (int) strlen(line); i++)
         {
-            if (line[i] != ' ') return FALSE;
+            if (line[i] != ' ')
+            {
+                return FALSE;
+            }
         }
     }
     return TRUE;
index 066888c..cfdff5f 100644 (file)
@@ -1,11 +1,11 @@
 /*
 * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
 * Copyright (C) 2010 - DIGITEO - Allan CORNET
-* 
+*
 * This file must be used under the terms of the CeCILL.
 * This source file is licensed as described in the file COPYING, which
 * you should have received as part of this distribution.  The terms
-* are also available at    
+* are also available at
 * http://www.cecill.info/licences/Licence_CeCILL_V2-en.txt
 *
 */
@@ -15,7 +15,8 @@
 
 #include "BOOL.h"
 
-typedef enum { 
+typedef enum
+{
     FSCANFMAT_NO_ERROR = 0,
     FSCANFMAT_MOPEN_ERROR = 1,
     FSCANFMAT_FORMAT_ERROR = 2,
@@ -25,7 +26,8 @@ typedef enum {
 } fscanfMatError;
 
 
-typedef struct {
+typedef struct
+{
     char **text;
     int sizeText;
     double *values;
@@ -41,7 +43,7 @@ typedef struct {
 /**
 *
 */
-fscanfMatResult *fscanfMat(char *filename, char *format, char *separator, BOOL asDouble);
+fscanfMatResult *fscanfMat(char *filename, char *format, char *separator);
 
 void freeFscanfMatResult(fscanfMatResult *resultStruct);