Fixed cppcheck errors [mockZMQ/DIET_client.cpp:248]: (style) Unused variable: serv [mockZMQ/DIET_client.cpp:354]: (style) Unused variable: vec [mockZMQ/Annuary.cpp:90]: (style) Unused variable: ports [UMS/src/server/UserServer.cpp:67]: (style) Unused variable: formatiduser [UMS/src/server/MachineServer.cpp:48]: (style) Unused variable: formatidmachine [FMS/test/src/fms_stress.cpp:103]: (style) Unused variable: i [IMS/src/cli/restart.cpp:42]: (style) Unused variable: vishnu [IMS/src/cli/stop.cpp:64]: (style) Unused variable: processName [IMS/test/src/ims_stress.cpp:59]: (style) Unused variable: i [TMS/src/client/WorkProxy.cpp:35]: (style) Unused variable: tmp [TMS/src/server/Env.cpp:43]: (style) Unused variable: numNodes [TMS/src/server/Env.cpp:44]: (style) Unused variable: cpuPerNode [TMS/src/server/Env.cpp:164]: (style) Unused variable: param [TMS/src/server/Env.cpp:167]: (style) Unused variable: pos1 [TMS/src/server/Env.cpp:167]: (style) Unused variable: pos2 [TMS/src/sed/tmssed.cpp:54]: (style) Unused variable: dietConfigFile [TMS/src/sed/internalApi.cpp:349]: (style) Unused variable: missingFiles [TMS/src/server/ScriptGenConvertor.cpp:511]: (style) Unused variable: tmp [TMS/src/server/WorkServer.cpp:47]: (style) Unused variable: formatidwork [TMS/src/server/TorqueServer.cpp:869]: (style) Unused variable: nextNode [TMS/src/server/TorqueServer.cpp:870]: (style) Unused variable: nbCpuStr [TMS/src/server/TorqueServer.cpp:1158]: (style) Unused variable: batchId [TMS/test/src/tms_stress.cpp:114]: (style) Unused variable: i [UMS/src/cli/list_machines.cpp:56]: (style) Unused variable: sessionKey
Coding standards + fixed cppcheck error: [FMS/src/server/FileTransferServer.cpp:531]: (style) The scope of the variable 'pid' can be reduced
Fixed cppcheck errors: [core/src/database/POSTGREDatabase.cpp:139]: (style) The scope of the variable 'res' can be reduced [core/src/database/POSTGREDatabase.cpp:143]: (style) The scope of the variable 'nFields' can be reduced [core/src/database/POSTGREDatabase.cpp:144]: (style) The scope of the variable 'i' can be reduced [core/src/database/POSTGREDatabase.cpp:145]: (style) The scope of the variable 'j' can be reduced

[core/src/utils/utilServer.cpp:304]: (style) The scope of the variable 'i' can be reduced
[core/src/utils/utilServer.cpp:578]: (style) The scope of the variable 'endOfLinePos' can be reduced

[mockZMQ/Server.cpp:94]: (style) The scope of the variable 'i' can be reduced

[core/src/utils/utilVishnu.cpp:932]: (style) The scope of the variable 'addPtr' can be reduced
[core/src/utils/utilVishnu.cpp:934]: (style) The scope of the variable 'addressBuffer' can be reduced

[FMS/src/cli/list_file_transfers.cpp:112]: (style) The scope of the variable 'status' can be reduced

[FMS/src/cli/fileTransferUtils.hpp:144]: (style) The scope of the variable 'trCmd' can be reduced
Fixed cppcheck errors: [mockZMQ/DIET_client.cpp:372]: (performance) Prefer prefix ++/-- operators for non-primitive types. [mockZMQ/Server.cpp:100]: (performance) Prefer prefix ++/-- operators for non-primitive types.
Coding standards Fixed cppcheck error: [TMS/src/server/Env.cpp:173]: (performance) Prefer prefix ++/-- operators for non-primitive types.
Added missing break in switch/case
Fixed cppcheck errors: [TMS/src/client/tmsClientUtils.cpp:125]: (performance) Prefer prefix ++/-- operators for non-primitive types. [core/src/utils/utilVishnu.cpp:754]: (performance) Prefer prefix ++/-- operators for non-primitive types. [core/src/utils/utilVishnu.cpp:765]: (performance) Prefer prefix ++/-- operators for non-primitive types.
coding standards
indentation
Fixed cppcheck errors: [mockZMQ/dispatcher.cpp:77]: (style) Exception should be caught by reference. [mockZMQ/dispatcher.cpp:129]: (style) Exception should be caught by reference.
Fixed cppcheck errors [mockZMQ/DIET_client.cpp:237]: (performance) Passing the result of c_str() to a function that takes std::string as argument 1 is slow and redundant. [mockZMQ/DIET_client.cpp:293]: (performance) Passing the result of c_str() to a function that takes std::string as argument 1 is slow and redundant.
Fixed cppcheck error [mockZMQ/SeD.cpp:61]: (style) Exception should be caught by reference.
Fixed cppcheck error [mockZMQ/zhelpers.hpp:102]: (style, inconclusive) Technically the member function 'LazyPirateClient::recv' can be const.
Fixed cppcheck errors: [mockZMQ/Server.cpp:45] -> [mockZMQ/Server.hpp:69]: (style, inconclusive) Technically the member function 'Server::hasService' can be const. [mockZMQ/Server.cpp:56] -> [mockZMQ/Server.hpp:76]: (style, inconclusive) Technically the member function 'Server::getName' can be const. [mockZMQ/Server.cpp:62] -> [mockZMQ/Server.hpp:90]: (style, inconclusive) Technically the member function 'Server::getURI' can be const.
Fixed cppcheck errors: [mockZMQ/Annuary.cpp:13]: (performance) Variable 'mservers' is assigned in constructor body. Consider performing initialization in initialization list. [mockZMQ/Annuary.cpp:17]: (performance) Variable 'mservers' is assigned in constructor body. Consider performing initialization in initialization list.
Fixed cppcheck error [core/src/version/VersionManager.cpp:45] -> [core/src/version/VersionManager.hpp:47]: (style, inconclusive) Technically the member function 'VersionManager::getError' can be const.
Fixed cppcheck error [core/src/utils/utilClient.hpp:47]: (performance) Function parameter 'msgComp' should be passed by reference. [core/src/utils/utilServer.hpp:277]: (performance) Function parameter 'msgComp' should be passed by reference.
Fixed cppcheck error [core/src/utils/QueryServer.hpp:37]: (warning) Member variable 'QueryServer::mparameters' is not initialized in the constructor.
Fixed cppcheck error [core/src/utils/cliUtil.hpp:28]: (style, inconclusive) Technically the member function 'env_name_mapper::operator()' can be const. + initialized variable
fixed cppcheck error: [core/src/utils/Options.hpp:278]: (performance) Function parameter 'key' should be passed by reference. + indent code