Fix some easy to fix warnings 10/17810/5
Clément DAVID [Tue, 16 Feb 2016 07:26:05 +0000 (08:26 +0100)]
The goal is to pass : make CXXFLAGS='-Wall -Werror' but some options are
still valid :

-Wno-sign-compare : to allow comparing vector.size() and int
-Wno-unused-parameter : to implement an API with ease

Change-Id: I9bb93dcb5d7754774745a9eca7cfa89afa039693

46 files changed:
scilab/modules/api_scilab/src/cpp/api_stack_common.cpp
scilab/modules/api_scilab/src/cpp/template/api_int.hpp
scilab/modules/ast/includes/analysis/TIType.hxx
scilab/modules/ast/includes/analysis/data/Block.hxx
scilab/modules/ast/includes/analysis/data/Info.hxx
scilab/modules/ast/includes/analysis/data/PolymorphicMacroCache.hxx
scilab/modules/ast/includes/analysis/gvn/SymbolicList.hxx
scilab/modules/ast/includes/analysis/gvn/TestGVNVisitor.hxx
scilab/modules/ast/includes/ast/breakpoint.hxx
scilab/modules/ast/includes/ast/debugmanager.hxx
scilab/modules/ast/includes/ast/printvisitor.hxx
scilab/modules/ast/src/cpp/analysis/PolymorphicMacroCache.cpp
scilab/modules/core/includes/storeCommand.h
scilab/modules/core/sci_gateway/cpp/sci_debug.cpp
scilab/modules/core/sci_gateway/cpp/sci_isfield.cpp
scilab/modules/core/sci_gateway/cpp/sci_who.cpp
scilab/modules/core/src/cpp/storeCommand.cpp
scilab/modules/elementary_functions/sci_gateway/cpp/sci_cumprod.cpp
scilab/modules/elementary_functions/sci_gateway/cpp/sci_cumsum.cpp
scilab/modules/elementary_functions/sci_gateway/cpp/sci_prod.cpp
scilab/modules/elementary_functions/sci_gateway/cpp/sci_sum.cpp
scilab/modules/fileio/sci_gateway/cpp/fileio_gw.cpp
scilab/modules/fileio/sci_gateway/cpp/sci_basename.cpp
scilab/modules/fileio/sci_gateway/cpp/sci_mfscanf.cpp
scilab/modules/fileio/sci_gateway/cpp/sci_mscanf.cpp
scilab/modules/fileio/sci_gateway/cpp/sci_msscanf.cpp
scilab/modules/fileio/src/cpp/scilab_sscanf.cpp
scilab/modules/functions/sci_gateway/cpp/sci_exec.cpp
scilab/modules/functions_manager/includes/funcmanager.hxx
scilab/modules/hdf5/src/cpp/H5AttributesList.cpp
scilab/modules/hdf5/src/cpp/H5AttributesList.hxx
scilab/modules/hdf5/src/cpp/H5CompoundData.hxx
scilab/modules/hdf5/src/cpp/H5Data.hxx
scilab/modules/hdf5/src/cpp/H5Group.cpp
scilab/modules/hdf5/src/cpp/H5Group.hxx
scilab/modules/hdf5/src/cpp/H5ListObject.hxx
scilab/modules/hdf5/src/cpp/H5NamedObjectsList.hxx
scilab/modules/hdf5/src/cpp/H5Object.hxx
scilab/modules/hdf5/src/cpp/H5Type.cpp
scilab/modules/history_manager/src/cpp/HistoryFile.hxx
scilab/modules/history_manager/src/cpp/HistoryManager.hxx
scilab/modules/history_manager/src/cpp/HistorySearch.hxx
scilab/modules/localization/sci_gateway/cpp/localization_gw.cpp
scilab/modules/signal_processing/sci_gateway/cpp/sci_corr.cpp
scilab/modules/sparse/sci_gateway/cpp/sci_luget.cpp
scilab/modules/string/sci_gateway/cpp/sci_strtok.cpp

index fda4144..785b091 100644 (file)
@@ -117,8 +117,6 @@ int returnArguments(void* _pvCtx)
 
 int checkInputArgument(void* _pvCtx, int _iMin, int _iMax)
 {
-    SciErr sciErr = sciErrInit();
-
     types::GatewayStruct *pStr = (types::GatewayStruct*)_pvCtx;
     int iRhs            = *getNbInputArgument(_pvCtx);
 
@@ -158,8 +156,6 @@ SciErr reshapeArray(void* _pvCtx, int* _piAddress, int* _iDimsArray, int _iDims)
 /*--------------------------------------------------------------------------*/
 int checkInputArgumentAtLeast(void* _pvCtx, int _iMin)
 {
-    SciErr sciErr = sciErrInit();
-
     if (_iMin <= nbInputArgument(_pvCtx))
     {
         return 1;
@@ -172,8 +168,6 @@ int checkInputArgumentAtLeast(void* _pvCtx, int _iMin)
 /*--------------------------------------------------------------------------*/
 int checkInputArgumentAtMost(void* _pvCtx, int _iMax)
 {
-    SciErr sciErr = sciErrInit();
-
     if (_iMax >= nbInputArgument(_pvCtx))
     {
         return 1;
@@ -186,8 +180,6 @@ int checkInputArgumentAtMost(void* _pvCtx, int _iMax)
 /*--------------------------------------------------------------------------*/
 int checkOutputArgument(void* _pvCtx, int _iMin, int _iMax)
 {
-    SciErr sciErr = sciErrInit();
-
     if (_iMin <= nbOutputArgument(_pvCtx) && _iMax >= nbOutputArgument(_pvCtx))
     {
         return 1;
@@ -208,8 +200,6 @@ int checkOutputArgument(void* _pvCtx, int _iMin, int _iMax)
 /*--------------------------------------------------------------------------*/
 int checkOutputArgumentAtLeast(void* _pvCtx, int _iMin)
 {
-    SciErr sciErr = sciErrInit();
-
     if (_iMin <= nbOutputArgument(_pvCtx))
     {
         return 1;
@@ -222,8 +212,6 @@ int checkOutputArgumentAtLeast(void* _pvCtx, int _iMin)
 /*--------------------------------------------------------------------------*/
 int checkOutputArgumentAtMost(void* _pvCtx, int _iMax)
 {
-    SciErr sciErr = sciErrInit();
-
     if (_iMax >= nbOutputArgument(_pvCtx))
     {
         return 1;
@@ -511,9 +499,9 @@ SciErr getVarType(void *_pvCtx, int *_piAddress, int *_piType)
         case types::InternalType::ScilabSparseBool :
             *_piType = sci_boolean_sparse;
             break;
-        //case types::InternalType::RealMatlabSparse :
-        //    *_piType = sci_matlab_sparse;
-        //    break;
+            //case types::InternalType::RealMatlabSparse :
+            //    *_piType = sci_matlab_sparse;
+            //    break;
         case types::InternalType::ScilabInt8 :
         case types::InternalType::ScilabUInt8 :
         case types::InternalType::ScilabInt16 :
index 66b8000..5169788 100644 (file)
@@ -40,9 +40,9 @@ int API_PROTO(getIntegerPrecision)(scilabEnv env, scilabVar var)
             return SCI_INT64;
         case types::InternalType::ScilabUInt64:
             return SCI_UINT64;
+        default:
+            return 0;
     }
-
-    return 0;
 }
 
 int API_PROTO(isInt8)(scilabEnv env, scilabVar var)
@@ -504,9 +504,9 @@ scilabStatus API_PROTO(getIntegerArray)(scilabEnv env, scilabVar var, void** val
             return scilab_getUnsignedInteger32Array(env, var, (unsigned int**)vals);
         case types::InternalType::ScilabUInt64:
             return scilab_getUnsignedInteger64Array(env, var, (unsigned long long**)vals);
+        default:
+            return STATUS_ERROR;
     }
-
-    return STATUS_ERROR;
 }
 
 scilabStatus API_PROTO(getInteger8Array)(scilabEnv env, scilabVar var, char** vals)
@@ -641,9 +641,10 @@ scilabStatus API_PROTO(setIntegerArray)(scilabEnv env, scilabVar var, const void
             return scilab_setUnsignedInteger32Array(env, var, (const unsigned int*)vals);
         case types::InternalType::ScilabUInt64:
             return scilab_setUnsignedInteger64Array(env, var, (const unsigned long long*)vals);
-    }
+        default:
+            return STATUS_ERROR;
 
-    return STATUS_ERROR;
+    }
 }
 
 scilabStatus API_PROTO(setInteger8Array)(scilabEnv env, scilabVar var, const char* vals)
index 62501ab..ad133a2 100644 (file)
@@ -811,7 +811,7 @@ namespace std
 template<>
 struct hash<analysis::TIType>
 {
-    inline size_t operator()(const analysis::TIType & t) const
+    inline size_t operator()(const analysis::TIType & /*t*/) const
     {
         return 0;//tools::hash_combine(t.type, t.rows, t.cols);
     }
index 1e37ccd..7fe4603 100644 (file)
@@ -58,7 +58,7 @@ public:
     };
 
     Block(DataManager * _dm);
-    Block(const unsigned int _id, Block * _parent, ast::Exp * _exp) : dm(_parent->dm), parent(_parent), exp(_exp), gvn(&_parent->getGVN()), isReturn(false), id(_id) { }
+    Block(const unsigned int _id, Block * _parent, ast::Exp * _exp) : dm(_parent->dm), exp(_exp), parent(_parent), symMap(), blocks(), gvn(&_parent->getGVN()), isReturn(false), id(_id) { }
 
     virtual ~Block();
 
index 1e5170c..7980489 100644 (file)
@@ -53,8 +53,8 @@ struct Info
     SymbolicRange range;
     SymbolicDimension maxIndex;
 
-    Info(Data * _data = nullptr) : R(false), W(false), O(false), isint(false), local(Local::INFO_TRUE), cleared(false), exists(true), data(_data), exp(nullptr) { }
-    Info(const Info & i) : R(i.R), W(i.W), O(i.O), isint(i.isint), local(i.local), cleared(i.cleared), exists(i.exists), constant(i.constant), range(i.range), maxIndex(i.maxIndex), type(i.type), data(i.data ? new Data(*i.data) : nullptr), exp(i.exp) { }
+    Info(Data * _data = nullptr) : R(false), W(false), O(false), isint(false), local(Local::INFO_TRUE), cleared(false), exists(true), type(), data(_data), exp(nullptr), constant(), range(), maxIndex() { }
+    Info(const Info & i) : R(i.R), W(i.W), O(i.O), isint(i.isint), local(i.local), cleared(i.cleared), exists(i.exists), type(i.type), data(i.data ? new Data(*i.data) : nullptr), exp(i.exp), constant(i.constant), range(i.range), maxIndex(i.maxIndex) { }
 
     void merge(const Info & info, bool & isSameData);
     void addData(const bool known, const symbol::Symbol & sym);
index 5732a18..ce4975b 100644 (file)
@@ -46,7 +46,7 @@ public:
     PolymorphicMacroCache();
     ~PolymorphicMacroCache();
 
-    const bool getOutTypes(AnalysisVisitor & visitor, MacroDef * macrodef, std::vector<TIType> & in, std::vector<TIType> & out, uint64_t & functionId);
+    bool getOutTypes(AnalysisVisitor & visitor, MacroDef * macrodef, std::vector<TIType> & in, std::vector<TIType> & out, uint64_t & functionId);
 
     friend std::wostream & operator<<(std::wostream & out, const PolymorphicMacroCache & pmc);
 
index 70d6702..692aace 100644 (file)
@@ -41,12 +41,12 @@ class SymbolicList
         Value(double val) : dval(val) { }
     };
 
+    bool symbolic;
+
     Value start;
     Value step;
     Value end;
 
-    bool symbolic;
-
 public:
 
     /**
index 8661c84..1d85644 100644 (file)
@@ -182,6 +182,24 @@ private:
                     setResult(gvn.getValue(int64_t(0)));
                 }
                 break;
+            case ast::OpExp::ldivide:
+            case ast::OpExp::dotldivide:
+            case ast::OpExp::krontimes:
+            case ast::OpExp::kronrdivide:
+            case ast::OpExp::kronldivide:
+            case ast::OpExp::controltimes:
+            case ast::OpExp::controlrdivide:
+            case ast::OpExp::controlldivide:
+            case ast::OpExp::lt:
+            case ast::OpExp::le:
+            case ast::OpExp::gt:
+            case ast::OpExp::ge:
+            case ast::OpExp::logicalAnd:
+            case ast::OpExp::logicalOr:
+            case ast::OpExp::logicalShortCutAnd:
+            case ast::OpExp::logicalShortCutOr:
+                std::cerr << "TestGVNVisitor: unsupported ast::OpExp" << std::endl;
+                break;
         }
     }
 
index 53766dd..2741de7 100644 (file)
@@ -27,8 +27,6 @@ struct Breakpoint
 {
     Breakpoint(const std::wstring& functionName, int iLine = -1, const std::wstring& condition = L"")
         : _pFunctionName(functionName), _iMacroLine(iLine), _file(L""), _iFileLine(0), _condition(condition), _conditionExp(NULL), enable(true) {}
-    //Breakpoint(std::wstring& file, int iLine = -1, ast::Exp* condition = NULL)
-    //    : _pFunctionName(NULL), _iMacroLine(iLine), _file(file), _condition(condition) {}
 
     ~Breakpoint()
     {
@@ -115,11 +113,11 @@ struct Breakpoint
 
 private:
     std::wstring _pFunctionName;
+    int _iMacroLine;
     std::wstring _file;
+    int _iFileLine;
     std::wstring _condition;
     ast::Exp* _conditionExp;
-    int _iFileLine;
-    int _iMacroLine;
     bool enable;
 };
 
index 1868db5..820b8e0 100644 (file)
@@ -41,8 +41,8 @@ private:
         breakpoints(),
         debuggers(), pExp(nullptr), interrupted(false), currentBreakPoint(-1), action(Continue), level(0) {}
 
-    Debuggers debuggers;
     Breakpoints breakpoints;
+    Debuggers debuggers;
 
     ast::Exp* pExp;
     bool interrupted;
index b1429ad..f0fb004 100644 (file)
@@ -127,10 +127,10 @@ public:
     `-------------*/
 protected:
     std::wostream   *ostr;
-    int             indent;
     bool            force_parenthesis;
     const bool      displayOriginal;
     const bool      headerOnly;
+    int             indent;
     bool            is_last_matrix_line;
     bool            is_last_column_comment;
 
index d583bac..206c3f8 100644 (file)
@@ -24,7 +24,7 @@ PolymorphicMacroCache::PolymorphicMacroCache() : id(1) { }
 
 PolymorphicMacroCache::~PolymorphicMacroCache() { }
 
-const bool PolymorphicMacroCache::getOutTypes(AnalysisVisitor & visitor, MacroDef * macrodef, std::vector<TIType> & in, std::vector<TIType> & out, uint64_t & functionId)
+bool PolymorphicMacroCache::getOutTypes(AnalysisVisitor & visitor, MacroDef * macrodef, std::vector<TIType> & in, std::vector<TIType> & out, uint64_t & functionId)
 {
     // TODO: handle varargin
     if (in.size() > macrodef->getRhs())
index 56629a4..556cddb 100644 (file)
@@ -33,7 +33,7 @@ enum command_origin_t
  * @param iCmdorigin        : origine of the command
  * @return <ReturnValue>
  */
-int StoreCommandWithFlags(char* command, int iPrioritary, int iInterruptible, enum command_origin_t iCmdorigin);
+int StoreCommandWithFlags(const char* command, int iPrioritary, int iInterruptible, enum command_origin_t iCmdorigin);
 
 /**
  * Store a non-prioritary and interruptible command
@@ -41,7 +41,7 @@ int StoreCommandWithFlags(char* command, int iPrioritary, int iInterruptible, en
  * @param command : the command
  * @return <ReturnValue>
  */
-int StoreCommand(char *command);
+int StoreCommand(const char *command);
 
 /**
  * Store a prioritary and interruptible command
@@ -50,14 +50,14 @@ int StoreCommand(char *command);
  * @return <ReturnValue>
  */
 
-int StoreConsoleCommand(char *command, int iWaitFor);
+int StoreConsoleCommand(const char *command, int iWaitFor);
 /**
  * Store a prioritary and non-interruptible command
  *
  * @param command : the command
  * @return <ReturnValue>
  */
-int StorePrioritaryCommand(char *command);
+int StorePrioritaryCommand(const char *command);
 
 /**
  * Get the next command to execute
index 94a0416..37f0c72 100644 (file)
@@ -51,6 +51,7 @@ enum EnumCommand
 };
 
 const char fname[] = "debug";
+const std::string HELP_DEBUG_COMMAND("help debug");
 
 void print_help();
 EnumCommand getCommand(const std::wstring& command);
@@ -297,7 +298,7 @@ types::Function::ReturnValue sci_debug(types::typed_list &in, int _iRetCount, ty
 
             if (ConfigVariable::getScilabMode() == SCILAB_NW || ConfigVariable::getScilabMode() == SCILAB_STD)
             {
-                StorePrioritaryCommand("help debug");
+                StorePrioritaryCommand(HELP_DEBUG_COMMAND.data());
                 return types::Function::OK;
             }
 
index 0160cfc..4ecc146 100644 (file)
@@ -81,6 +81,9 @@ types::Function::ReturnValue sci_isfield(types::typed_list &in, int _iRetCount,
             }
             break;
         }
+        default:
+            // never occurs
+            return types::Function::Error;
     }
 
     out.push_back(pOutBool);
index a5c6520..c6a8d34 100644 (file)
@@ -30,14 +30,14 @@ extern "C"
 
 types::Function::ReturnValue sci_who(types::typed_list& in, int _iRetCount, types::typed_list& out)
 {
-    wchar_t* wcsWhat = L"";
+    std::wstring wcsWhat(L"");
     bool bSorted = false;
     types::String* pStrOut = NULL;
     types::Double* pDblOut = NULL;
     std::list<std::wstring> lstVar;
     int size = 0;
 
-    if (in.size() < 0 || in.size() > 2)
+    if (in.size() > 2)
     {
         Scierror(77, _("%s: Wrong number of input argument(s): %d to %d expected.\n"), "who", 0, 2);
         return types::Function::Error;
@@ -104,15 +104,15 @@ types::Function::ReturnValue sci_who(types::typed_list& in, int _iRetCount, type
         wcsWhat = pStrWhat->get(0);
     }
 
-    if (wcscmp(wcsWhat, L"local") == 0 || wcscmp(wcsWhat, L"get") == 0)
+    if (wcsWhat == L"local" || wcsWhat == L"get")
     {
         size = symbol::Context::getInstance()->getVarsNameForWho(lstVar, bSorted);
     }
-    else if (wcscmp(wcsWhat, L"global") == 0)
+    else if (wcsWhat == L"global")
     {
         size = symbol::Context::getInstance()->getGlobalNameForWho(lstVar, bSorted);
     }
-    else if (bSorted == false && wcscmp(wcsWhat, L"sorted") == 0)
+    else if (bSorted == false && wcsWhat == L"sorted")
     {
         bSorted = true;
         std::wstringstream wstream;
index 7d71040..e1b3e86 100644 (file)
@@ -51,7 +51,7 @@ struct CommandRec
 static std::list<CommandRec> commandQueue;
 static std::list<CommandRec> commandQueuePrioritary;
 /*--------------------------------------------------------------------------*/
-int StoreCommandWithFlags(char* command, int iPrioritary, int iInterruptible, command_origin_t iCmdOrigin)
+int StoreCommandWithFlags(const char* command, int iPrioritary, int iInterruptible, command_origin_t iCmdOrigin)
 {
     ThreadManagement::LockStoreCommand();
     if (iPrioritary)
@@ -73,7 +73,7 @@ int StoreCommandWithFlags(char* command, int iPrioritary, int iInterruptible, co
     return 0;
 }
 
-int StoreCommand(char *command)
+int StoreCommand(const char *command)
 {
     ThreadManagement::LockStoreCommand();
     commandQueue.emplace_back(os_strdup(command),
@@ -88,7 +88,7 @@ int StoreCommand(char *command)
     return 0;
 }
 
-int StoreConsoleCommand(char *command, int iWaitFor)
+int StoreConsoleCommand(const char *command, int iWaitFor)
 {
     ThreadManagement::LockStoreCommand();
     commandQueuePrioritary.emplace_back(os_strdup(command),
@@ -116,7 +116,7 @@ int StoreConsoleCommand(char *command, int iWaitFor)
     return 0;
 }
 
-int StorePrioritaryCommand(char *command)
+int StorePrioritaryCommand(const char *command)
 {
     ThreadManagement::LockStoreCommand();
     commandQueuePrioritary.emplace_back(os_strdup(command),
index b72b72c..85576b9 100644 (file)
@@ -381,6 +381,8 @@ types::Function::ReturnValue sci_cumprod(types::typed_list &in, int _iRetCount,
                 out.push_back(toInt<types::UInt64>(pDblOut));
                 break;
             }
+            default:
+                return types::Function::Error;
         }
 
         if (pDblOut)
index 4328872..809bff7 100644 (file)
@@ -385,6 +385,8 @@ types::Function::ReturnValue sci_cumsum(types::typed_list &in, int _iRetCount, t
                 out.push_back(toInt<types::UInt64>(pDblOut));
                 break;
             }
+            default:
+                return types::Function::Error;
         }
 
         if (pDblOut)
index 080f40d..ba097d5 100644 (file)
@@ -423,6 +423,8 @@ types::Function::ReturnValue sci_prod(types::typed_list &in, int _iRetCount, typ
                 out.push_back(toInt<types::UInt64>(pDblOut));
                 break;
             }
+            default:
+                return types::Function::Error;
         }
 
         if (pDblOut)
index b2eb3d7..462ba3a 100644 (file)
@@ -419,6 +419,8 @@ types::Function::ReturnValue sci_sum(types::typed_list &in, int _iRetCount, type
                 out.push_back(toInt<types::UInt64>(pDblOut));
                 break;
             }
+            default:
+                return types::Function::Error;
         }
 
         if (pDblOut)
index 7918e55..94f4e32 100644 (file)
 
 #define MODULE_NAME L"fileio"
 
+#ifdef _MSC_VER
 #pragma comment(lib,"../../../../bin/libxml2.lib")
 #pragma comment(lib,"../../../../bin/libcurl.lib")
+#endif
 
 extern "C"
 {
index 10ec4fc..74c1841 100644 (file)
@@ -30,7 +30,6 @@ extern "C"
 types::Function::ReturnValue sci_basename(types::typed_list &in, int _iRetCount, types::typed_list &out)
 {
     int iExpand     = 1;
-    int iConvert    = 1;
 
     if (in.size() < 1 || in.size() > 3)
     {
@@ -74,8 +73,6 @@ types::Function::ReturnValue sci_basename(types::typed_list &in, int _iRetCount,
             Scierror(999, _("%s: Wrong size for input argument #%d: A scalar boolean expected.\n"), "basename", 2);
             return types::Function::Error;
         }
-
-        iConvert = in[1]->getAs<types::Bool>()->get()[0];
     }
 
     if (in[0]->isDouble() && in[0]->getAs<types::Double>()->isEmpty())
index fec751d..0c74712 100644 (file)
@@ -212,6 +212,8 @@ types::Function::ReturnValue sci_mfscanf(types::typed_list &in, int _iRetCount,
                 uiFormatUsed |= (1 << 2);
                 break;
             }
+            case NONE:
+                break;
         }
     }
 
index 2bec804..54a13ec 100644 (file)
@@ -175,6 +175,8 @@ types::Function::ReturnValue sci_mscanf(types::typed_list &in, int _iRetCount, t
                 uiFormatUsed |= (1 << 2);
             }
             break;
+            case NONE:
+                break;
         }
     }
 
index 92e26d0..8a7589e 100644 (file)
@@ -159,6 +159,8 @@ types::Function::ReturnValue sci_msscanf(types::typed_list &in, int _iRetCount,
                 uiFormatUsed |= (1 << 2);
             }
             break;
+            case NONE:
+                break;
         }
     }
 
index 80d221c..9baeace 100644 (file)
@@ -41,7 +41,6 @@ int scilab_sscanf(wchar_t* _wcsFormat, wchar_t* _wcsData, int _iIterrator, int _
     int dimsArray[2]            = {_iNiter, 1};
     BOOL bStar                  = FALSE;
     BOOL bUnsigned              = FALSE;
-    BOOL bNegatif               = FALSE;
     BOOL bIgnoredChars          = TRUE;
     int base                    = 0;
     wchar_t wcsLLH              = L' ';
@@ -576,7 +575,6 @@ int scilab_sscanf(wchar_t* _wcsFormat, wchar_t* _wcsData, int _iIterrator, int _
                         wcsLLH          = L' ';
                         bIgnoredChars   = TRUE;
                         bUnsigned       = FALSE;
-                        bNegatif        = FALSE;
                         bStar           = FALSE;
                         base            = 0;
                         i++;
index 74d71e7..02b2f8b 100644 (file)
@@ -71,7 +71,6 @@ void closeFile(std::ifstream* file, int fileId, const std::wstring& wstFile, ast
 types::Function::ReturnValue sci_exec(types::typed_list &in, int _iRetCount, types::typed_list &out)
 {
     int promptMode      = 0;//default value at startup, overthise 3 or verbose ";"
-    bool bPromptMode    = false;
     int iErr            = 0;
     bool bErrCatch      = false;
     ast::Exp* pExp      = NULL;
@@ -133,7 +132,6 @@ types::Function::ReturnValue sci_exec(types::typed_list &in, int _iRetCount, typ
                 }
 
                 promptMode = (int)in[2]->getAs<types::Double>()->getReal()[0];
-                bPromptMode = true;
             }
         }
         else if (in[1]->isDouble() && in[1]->getAs<types::Double>()->isScalar())
@@ -145,7 +143,6 @@ types::Function::ReturnValue sci_exec(types::typed_list &in, int _iRetCount, typ
             }
             //mode
             promptMode = (int)in[1]->getAs<types::Double>()->getReal()[0];
-            bPromptMode = true;
         }
         else
         {
@@ -261,8 +258,8 @@ types::Function::ReturnValue sci_exec(types::typed_list &in, int _iRetCount, typ
         // unable for macro with varargin or varargout
         auto inputs = pMacro->getInputs();
         auto outputs = pMacro->getOutputs();
-        if ((inputs->size() != 0 && inputs->back()->getSymbol().getName() == L"varargin") ||
-                outputs->size() != 0 && outputs->back()->getSymbol().getName() == L"varargout")
+        if ((inputs->size() != 0 && (inputs->back()->getSymbol().getName() == L"varargin")) ||
+                (outputs->size() != 0 && (outputs->back()->getSymbol().getName() == L"varargout")))
         {
             Scierror(999, _("%s: Wrong type for input argument #%d: A macro without varargin and varargout expected.\n"), "exec", 1);
             return types::Function::Error;
index d65471b..db0861a 100644 (file)
@@ -22,7 +22,9 @@
 #include <string>
 
 //disable warnings about exports STL items
+#ifdef _MSC_VER
 #pragma warning (disable : 4251)
+#endif
 
 #ifdef _MSC_VER
 #if FUNC_MAN_EXPORTS
index c4ddcba..982808b 100644 (file)
@@ -25,7 +25,7 @@ H5AttributesList::H5AttributesList(H5Object & _parent, const unsigned int _size,
 
 H5AttributesList::~H5AttributesList() { }
 
-const unsigned int H5AttributesList::getSize() const
+unsigned int H5AttributesList::getSize() const
 {
     if (H5ListObject<H5Attribute>::indexList)
     {
index 557a574..3ebbdfa 100644 (file)
@@ -35,7 +35,7 @@ public :
     virtual void setObject(const unsigned int pos, H5Attribute & attribute);
     virtual H5Attribute & getObject(const int pos);
     virtual H5Attribute & getObject(const std::string & name);
-    virtual const unsigned int getSize() const;
+    virtual unsigned int getSize() const;
 
     virtual std::string dump(std::map<haddr_t, std::string> & alreadyVisited, const unsigned int indentLevel) const;
     virtual std::string toString(const unsigned int indentLevel) const;
index 2c6514b..e9338b4 100644 (file)
@@ -76,11 +76,11 @@ protected:
 
 private:
 
+    const hid_t type;
     const hsize_t * cumprod;
     unsigned int nfields;
     std::map<std::string, FieldInfo *> * infos;
     FieldInfo ** fieldinfos;
-    const hid_t type;
 };
 }
 
index 111fa84..e1461b5 100644 (file)
@@ -26,11 +26,11 @@ class H5Data : public H5Object
 
 protected:
 
-    void * data;
     const hsize_t totalSize;
     const hsize_t dataSize;
     const hsize_t ndims;
     const hsize_t * dims;
+    void * data;
     const hsize_t stride;
     const size_t offset;
     const bool dataOwner;
index a376346..988fb8c 100644 (file)
@@ -92,7 +92,7 @@ H5TypesList & H5Group::getTypes()
     return *new H5TypesList(*this);
 }
 
-const unsigned int H5Group::getLinksSize() const
+unsigned int H5Group::getLinksSize() const
 {
     herr_t err;
     H5G_info_t info;
index d8f13bd..5d6e3d7 100644 (file)
@@ -69,7 +69,7 @@ public:
     virtual H5NamedObjectsList<H5Dataset> & getHardDatasets();
     virtual H5DatasetsList & getDatasets();
     virtual H5TypesList & getTypes();
-    const unsigned int getLinksSize() const;
+    unsigned int getLinksSize() const;
     virtual std::string getCompletePath() const;
     virtual std::string toString(const unsigned int indentLevel) const;
     virtual std::string dump(std::map<haddr_t, std::string> & alreadyVisited, const unsigned int indentLevel = 0) const;
index 99eee3d..e598388 100644 (file)
@@ -51,7 +51,7 @@ public :
         throw H5Exception(__LINE__, __FILE__, _("Invalid operation"));
     }
 
-    virtual const unsigned int getSize() const = 0;
+    virtual unsigned int getSize() const = 0;
 
     virtual void getAccessibleAttribute(const double index, const int pos, void * pvApiCtx) const
     {
index 1edcb0a..3b613e1 100644 (file)
@@ -43,8 +43,6 @@ class H5NamedObjectsList : public H5ListObject<T>
         int basetype;
     } OpData;
 
-    int prevPos;
-    hsize_t idx;
 
 public :
 
@@ -162,12 +160,12 @@ public :
         return (herr_t)0;
     }
 
-    const unsigned int getSize() const
+    unsigned int getSize() const
     {
         return getSize(false);
     }
 
-    const unsigned int getSize(const bool indexChecking) const
+    unsigned int getSize(const bool indexChecking) const
     {
         if (H5ListObject<T>::indexList && !indexChecking)
         {
@@ -244,6 +242,12 @@ protected :
     const int linkType;
     const std::string baseTypeName;
 
+private :
+
+    int prevPos;
+    hsize_t idx;
+
+
 private:
 
     T & getObject(const int pos, const bool checkPos)
index 8bc3784..7208c33 100644 (file)
@@ -54,14 +54,19 @@ class H5Object
 {
     static H5Object* root;
 
-    bool locked;
     H5Object & parent;
+    std::set<H5Object *> children;
+    bool locked;
     int scilabId;
 
     friend class H5AttributesList;
     friend class H5LinkList;
     friend class H5Dataset;
 
+protected: // for error report only
+
+    const std::string name;
+
 public :
 
     enum FilterType {HARD, SOFT, EXTERNAL, DANGLING, GROUP, DATASET, TYPE, ATTRIBUTE};
@@ -257,7 +262,7 @@ public :
         scilabId = id;
     }
 
-    const int getScilabId() const
+    int getScilabId() const
     {
         return scilabId;
     }
@@ -401,8 +406,6 @@ protected :
         OpDataSoftLinkFilter(std::vector<std::string> * _name, std::vector<std::string> * _value, FilterType _type) : name(_name), value(_value), type(_type) { }
     };
 
-    const std::string name;
-    std::set<H5Object *> children;
     void registerChild(H5Object * child)
     {
         if (!locked)
index 97a795f..033b7e2 100644 (file)
@@ -620,7 +620,6 @@ std::string H5Type::getNameFromType(hid_t type)
             break;
         case H5T_STRING:
             return "H5T_STRING";
-            break;
         case H5T_BITFIELD:
             if (H5Tequal(type, H5T_STD_B8BE) > 0)
             {
@@ -719,7 +718,6 @@ std::string H5Type::getNameFromType(hid_t type)
             {
                 return "H5T_STD_REF_OBJECT";
             }
-            break;
         case H5T_ENUM:
             return "H5T_ENUM";
         case H5T_VLEN:
@@ -729,6 +727,7 @@ std::string H5Type::getNameFromType(hid_t type)
         default:
             return _("Unknown datatype");
     }
+    return _("Unknown datatype");
 }
 
 void H5Type::printComplexNameFromType(std::ostringstream & os, const unsigned int indentLevel, const hid_t type)
index f17a412..1c7bbb9 100644 (file)
@@ -23,8 +23,10 @@ extern "C"
 #include "BOOL.h"
 }
 
+#ifdef _MSC_VER
 //disable warnings about exports STL items
 #pragma warning (disable : 4251)
+#endif
 
 /*------------------------------------------------------------------------*/
 typedef enum
index f401ce2..251af27 100644 (file)
@@ -30,8 +30,10 @@ extern "C"
 #include "HistoryFile.hxx"
 #include "HistorySearch.hxx"
 
+#ifdef _MSC_VER
 //disable warnings about exports STL items
 #pragma warning (disable : 4251)
+#endif
 
 /*------------------------------------------------------------------------*/
 class HISTORY_MANAGER_IMPEXP HistoryManager
index 3733e26..d19abc3 100644 (file)
@@ -23,8 +23,10 @@ extern "C"
 #include "BOOL.h"
 }
 
+#ifdef _MSC_VER
 //disable warnings about exports STL items
 #pragma warning (disable : 4251)
+#endif
 
 /*------------------------------------------------------------------------*/
 class HISTORY_MANAGER_IMPEXP HistorySearch
index 9fa8228..3830a6f 100644 (file)
@@ -17,7 +17,9 @@
 #include "localization_gw.hxx"
 #include "context.hxx"
 
+#ifdef _MSC_VER
 #pragma comment(lib,"../../../../bin/libintl.lib")
+#endif
 
 extern "C"
 {
index f966e29..3b76113 100644 (file)
@@ -48,8 +48,6 @@ types::Function::ReturnValue sci_corr(types::typed_list &in, int _iRetCount, typ
         if (pS->getSize() == 1 && pS->get(0)[0] == L'f')
         {
             //[cov,mean]=corr('fft',xmacro,[ymacro],n,sect)
-            types::InternalType* pXFunction = NULL;
-            types::InternalType* pYFunction = NULL;
 
             int iErr        = 0;
             int iSect       = 0;
@@ -102,12 +100,10 @@ types::Function::ReturnValue sci_corr(types::typed_list &in, int _iRetCount, typ
             //get xmacro
             if (in[1]->isCallable())
             {
-                pXFunction = in[1]->getAs<types::Callable>();
                 spFunctionsManager->setDgetx(in[1]->getAs<types::Callable>());
             }
             else if (in[1]->isString())
             {
-                pXFunction = in[1]->getAs<types::String>();
                 spFunctionsManager->setDgetx(in[1]->getAs<types::String>());
             }
             else
@@ -123,12 +119,10 @@ types::Function::ReturnValue sci_corr(types::typed_list &in, int _iRetCount, typ
                 //get ymacro
                 if (in[2]->isCallable())
                 {
-                    pYFunction = in[2]->getAs<types::Callable>();
                     spFunctionsManager->setDgety(in[2]->getAs<types::Callable>());
                 }
                 else if (in[2]->isString())
                 {
-                    pYFunction = in[2]->getAs<types::String>();
                     spFunctionsManager->setDgety(in[2]->getAs<types::String>());
                 }
                 else
index 7c3ad32..ad8106d 100644 (file)
@@ -35,7 +35,6 @@ types::Function::ReturnValue sci_luget(types::typed_list &in, int _iRetCount, ty
     double reltol   = 0.001;
     int nrank       = 0;
     int ierr        = 0;
-    int m1          = 0;
     int n1          = 0;
     int nl          = 0;
     int nu          = 0;
@@ -79,7 +78,6 @@ types::Function::ReturnValue sci_luget(types::typed_list &in, int _iRetCount, ty
     }
 
     types::Pointer *pPointerIn = in[0]->getAs<types::Pointer>();
-    m1 = pPointerIn->getRows();
     n1 = pPointerIn->getCols();
     cplx = pPointerIn->isComplex();
     pData = pPointerIn->get();
index 59019d3..b15eafc 100644 (file)
@@ -30,6 +30,7 @@ extern "C"
 }
 
 static wchar_t *pwstState = NULL;
+static /* const */ wchar_t EOS = 0;
 
 types::Function::ReturnValue sci_strtok(types::typed_list &in, int _iRetCount, types::typed_list &out)
 {
@@ -85,7 +86,8 @@ types::Function::ReturnValue sci_strtok(types::typed_list &in, int _iRetCount, t
     wchar_t* pwstToken = NULL;
     if (pwstString == NULL && pwstState == NULL)
     {
-        pwstToken = L"";
+        // set an empty string
+        pwstToken = &EOS;
     }
     else
     {