diff --git a/silecs-communication-cpp/src/silecs-communication/interface/communication/MBConnection.cpp b/silecs-communication-cpp/src/silecs-communication/interface/communication/MBConnection.cpp
index a21bba42ad91cc432af825d6e7599f4c7f3bdb3a..cc457dd449a0b8f297cec43db21016db2a5a7b36 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/communication/MBConnection.cpp
+++ b/silecs-communication-cpp/src/silecs-communication/interface/communication/MBConnection.cpp
@@ -195,7 +195,7 @@ int MBConnection::writeCoils(modbus_t *ctx, long start_addr, uint16_t count, uin
// The src array must contain bytes set to TRUE or FALSE. We need one byte for each bit in data, e.g 2 bytes would require a 16 byte bool array
uint8_t src[count * 8];
uint16_t bit_count;
- string tmp;
+ std::string tmp;
int i, k;
int j = 0;
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/PLCRegister.cpp b/silecs-communication-cpp/src/silecs-communication/interface/equipment/PLCRegister.cpp
index 239522fd73d9bf03b86373ac6fbd82d7d78761d0..321c36e9d2f67afe503d7054a8c594451cdd7b0d 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/PLCRegister.cpp
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/PLCRegister.cpp
@@ -542,7 +542,7 @@ void UnityRegister::importString(void* pBuffer, timeval ts)
for (unsigned long i = 0; i < (dimension1_ * dimension2_); i++) // scalar has dim1=dim2=1
{
unsigned char* pData = & (pDataBase[i * step]);
- size_t strSize = min((size_t)length_, strlen((const char*)pData));
+ size_t strSize = std::min((size_t)length_, strlen((const char*)pData));
pRecvStringValue_[i]->assign((const char*)pData, strSize);
}
tod_ = ts;
@@ -569,7 +569,7 @@ void UnityRegister::exportString(void* pBuffer)
for (unsigned long i = 0; i < dimension1_ * dimension2_; i++) // scalar has dim1=dim2=1
{ //Current size of the sent string should not exceed the register string max length
size_t strSize = pSendStringValue_[i]->size();
- std::string sendStringValue = pSendStringValue_[i]->substr(0, min((size_t)length_, strSize));
+ std::string sendStringValue = pSendStringValue_[i]->substr(0, std::min((size_t)length_, strSize));
bzero((void *)& (pData[i * step]), step); //TwinCAT string requires '\0' terminator (mem-size of the string (~step) has been properly dimensioned in the mapping).
memcpy((void *)& (pData[i * step]), (void *)sendStringValue.c_str(), sendStringValue.size());
}
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.cpp b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.cpp
index 66f4ec4cf4b575c68f7c501be3c3576e04d4184f..bf6218f4f3162b9944d79a39f964f2f966b04ee0 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.cpp
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.cpp
@@ -150,7 +150,7 @@ std::string Cluster::getBlockList()
return blockList;
}
-PLC* Cluster::getPLC(std::string plcID, string parameterFile)
+PLC* Cluster::getPLC(std::string plcID, std::string parameterFile)
{
//getPLC() supports IP and host-name addressing, but both are required to register the PLC object:
//Host-name will be used to manage related configuration documents.
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.h b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.h
index 386a3043a3ceec30551e166d4935af51e27cd9c0..6efddd50d13675026c8909711cbcb5a715970e02 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.h
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsCluster.h
@@ -124,7 +124,7 @@ public:
* \param parameterFile optional parameterFile which should be used to configure this PLC
* \return Reference of the PLC object
*/
- PLC* getPLC(std::string plcID, string parameterFile = "");
+ PLC* getPLC(std::string plcID, std::string parameterFile = "");
/*!
* \brief Provides the list of the PLCs attached to the Cluster (using getPLC() method)
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.cpp b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.cpp
index 01e0f6cfaa74aa101dca8b50c768cd29f2c58527..a221918e131358a2f1405a6cc4a92d976345fe54 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.cpp
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.cpp
@@ -39,7 +39,7 @@ Contributors:
namespace Silecs
{
connMapType PLC::connMap_;
-PLC::PLC(Cluster* theCluster, std::string plcName, std::string plcIPaddr, string parameterFile) :
+PLC::PLC(Cluster* theCluster, std::string plcName, std::string plcIPaddr, std::string parameterFile) :
theCluster_(theCluster),
IPaddr_(plcIPaddr),
parameterFile_(parameterFile)
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.h b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.h
index 0386fc32d28a59380dab8da11f37c91591b0b2f0..118c0390871b050f710b39e09abb76b0c1830f82 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.h
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsPLC.h
@@ -479,7 +479,7 @@ private:
friend class CNVSendBlockMode;
friend class CNVSendDeviceMode;
- PLC(Cluster* theCluster, std::string plcName, std::string plcIPaddr, string parameterFile = "");
+ PLC(Cluster* theCluster, std::string plcName, std::string plcIPaddr, std::string parameterFile = "");
virtual ~PLC();
inline PLCType& getTypeID()
@@ -592,7 +592,7 @@ private:
long baseAOAddr_;
std::string usedMem_;
- string parameterFile_;
+ std::string parameterFile_;
// Unique instance of the Header Device for diagnostic purpose
Device* theHeader_;
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsRegister.cpp b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsRegister.cpp
index 4c66a67f2b987bf8919548128d9c2e4b204b7032..a0d9022921705c9291868ca249a829de29df657c 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsRegister.cpp
+++ b/silecs-communication-cpp/src/silecs-communication/interface/equipment/SilecsRegister.cpp
@@ -1597,7 +1597,7 @@ std::string Register::dumpInputVal(bool asAsciiChar)
os << (char) ((unsigned char*)pRecvValue_)[i];
}
else
- os << hex << (unsigned int) ((unsigned char*)pRecvValue_)[i] << " ";
+ os << std::hex << (unsigned int) ((unsigned char*)pRecvValue_)[i] << " ";
}
return os.str();
}
@@ -1611,7 +1611,7 @@ std::string Register::dumpOutputVal(bool asAsciiChar)
if (asAsciiChar)
os << (char) ((unsigned char*)pSendValue_)[i];
else
- os << hex << (unsigned int) ((unsigned char*)pSendValue_)[i] << " ";
+ os << std::hex << (unsigned int) ((unsigned char*)pSendValue_)[i] << " ";
}
return os.str();
}
diff --git a/silecs-communication-cpp/src/silecs-communication/interface/utility/Thread.h b/silecs-communication-cpp/src/silecs-communication/interface/utility/Thread.h
index 82b17c4dcda29f389e6250920d70e668df40ba67..15718aa1eedebd79b35a894ad4a7f0e28032b5e0 100644
--- a/silecs-communication-cpp/src/silecs-communication/interface/utility/Thread.h
+++ b/silecs-communication-cpp/src/silecs-communication/interface/utility/Thread.h
@@ -15,6 +15,7 @@ Contributors:
#define _THREAD_H_
#include <iostream>
+#include <cstring>
#include <string>
#include <vector>
#include <list>
@@ -24,13 +25,10 @@ Contributors:
#include <pthread.h>
#include <signal.h>
#include <malloc.h>
-#include <string.h>
#include <silecs-communication/interface/core/Context.h>
#include <silecs-communication/interface/utility/Condition.h>
-using namespace std;
-
namespace Silecs
{
class Context;
@@ -184,7 +182,7 @@ unsigned int Thread<UserDataType>::getPriority()
if (pthread_getschedparam(*pTid_, &policy, ¶ms) != 0)
{
- string errMsg(string("Cannot get dynamically schedparams: ") + strerror(errno));
+ std::string errMsg(std::string("Cannot get dynamically schedparams: ") + std::strerror(errno));
throw errMsg;
}
return params.sched_priority;
@@ -202,7 +200,7 @@ void Thread<UserDataType>::setPriority(unsigned int rtprio)
if (pthread_setschedparam(*pTid_, sched_getscheduler(getpid()), ¶ms) == 0)
return;
}
- string errMsg(string("Cannot set dynamically schedparams: ") + strerror(errno));
+ std::string errMsg(std::string("Cannot set dynamically schedparams: ") + std::strerror(errno));
throw errMsg;
}
diff --git a/silecs-diagnostic-cpp/src/silecs-diagnostic/diagnostictoolmainview.cpp b/silecs-diagnostic-cpp/src/silecs-diagnostic/diagnostictoolmainview.cpp
index 03ae9455f02f46997b24bfb9a81e986961582298..ceabd3a3c36213b917c8229ccb3d22d2f6176d8b 100755
--- a/silecs-diagnostic-cpp/src/silecs-diagnostic/diagnostictoolmainview.cpp
+++ b/silecs-diagnostic-cpp/src/silecs-diagnostic/diagnostictoolmainview.cpp
@@ -156,7 +156,7 @@ void diagnosticToolMainView::loadFiles()
std::string writeBlockList = cluster->getBlockList(AccessType::Command)+" "+cluster->getBlockList(AccessType::Setting);
- istringstream blockListSplitted(writeBlockList);
+ std::istringstream blockListSplitted(writeBlockList);
Utils::logInfo(ui->console, writeBlockList);
do
{
@@ -170,7 +170,7 @@ void diagnosticToolMainView::loadFiles()
// Receive combo box
std::string readBlockList = cluster->getBlockList(AccessType::Acquisition)+" "+cluster->getBlockList(AccessType::Setting);
- istringstream blockListSplitted2(readBlockList);
+ std::istringstream blockListSplitted2(readBlockList);
Utils::logInfo(ui->console,readBlockList);
do
{
@@ -184,7 +184,7 @@ void diagnosticToolMainView::loadFiles()
// Send combo box
std::string copyBlockList = cluster->getBlockList(AccessType::Setting);
- istringstream blockListSplitted3(copyBlockList);
+ std::istringstream blockListSplitted3(copyBlockList);
Utils::logInfo(ui->console, copyBlockList);
do
{
diff --git a/silecs-diagnostic-cpp/src/silecs-diagnostic/loginhandler.h b/silecs-diagnostic-cpp/src/silecs-diagnostic/loginhandler.h
index 4824cefed08ed2d85ffe5e78cfffae02ab0934ed..c5fd44a020d9b27af4adf04ad912cb2cfe1aac7d 100755
--- a/silecs-diagnostic-cpp/src/silecs-diagnostic/loginhandler.h
+++ b/silecs-diagnostic-cpp/src/silecs-diagnostic/loginhandler.h
@@ -24,8 +24,6 @@ Contributors:
#include <cmw-rbac/AuthenticationFailure.h>
#endif
-using namespace std;
-
#ifdef WITH_RBAC
extern std::string USER_NAME;
diff --git a/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.cpp b/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.cpp
index 3ecf3208d26547aa66f553c25fc8984c86ef460d..9b8d83a779426b4aa85e05cdfea83fcd866cef64 100755
--- a/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.cpp
+++ b/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.cpp
@@ -102,7 +102,7 @@ void silecsModule::updatePLCRunState(Item *plcItem)
}
-Item *silecsModule::generateTree(string className, string deployFile)
+Item *silecsModule::generateTree(const std::string& className, const std::string& deployFile)
{
Utils::logInfo(messageConsole_,"Loading deploy file: " + deployFile);
XMLParser parserDeploy(deployFile,true);
@@ -116,7 +116,7 @@ Item *silecsModule::generateTree(string className, string deployFile)
else
{
ElementXML silecsDesign = parserDeploy.getFirstElementFromXPath("/SILECS-Deploy/SilecsDesign[@silecs-design-name='" + className + "']");
- string classVersion = silecsDesign.getAttribute("silecs-design-version");
+ std::string classVersion = silecsDesign.getAttribute("silecs-design-version");
silecsCluster = silecsService->getCluster(className,classVersion);
root->setText(0,QString::fromStdString(className+" v"+classVersion));
}
@@ -124,15 +124,15 @@ Item *silecsModule::generateTree(string className, string deployFile)
root->setLinkedObject(silecsCluster);
ElementXML deployUnitNode = parserDeploy.getFirstElementFromXPath("/SILECS-Deploy/Deploy-Unit");
- string deployName = deployUnitNode.getAttribute("name");
+ std::string deployName = deployUnitNode.getAttribute("name");
std::size_t deployFolderPos = deployFile.find(deployName);
- string deployProjectPath = deployFile.substr(0,deployFolderPos) + deployName;
+ std::string deployProjectPath = deployFile.substr(0,deployFolderPos) + deployName;
std::vector<ElementXML> controllerNodes = parserDeploy.getElementsFromXPath("/SILECS-Deploy/Controller");
for(auto controllerIter = controllerNodes.begin(); controllerIter != controllerNodes.end(); controllerIter++)
{
std::string plcName = controllerIter->getAttribute("host-name");
Silecs::PLC *plc;
- string parameterFile = "";
+ std::string parameterFile = "";
try
{
parameterFile = deployProjectPath + "/generated-silecs/client/" + plcName + ".silecsparam";
@@ -171,7 +171,7 @@ Item *silecsModule::generateTree(string className, string deployFile)
// get register List for the current device
std::string registerList = device->getRegisterList();
- istringstream registerListSplitted(registerList);
+ std::istringstream registerListSplitted(registerList);
do
{
@@ -212,7 +212,7 @@ void silecsModule::setScalarDataInDeviceFromItem(Item *currentItem, std::string
std::string deviceName = device->getLabel();
std::string registerList = device->getRegisterList();
- istringstream registerListSplitted(registerList);
+ std::istringstream registerListSplitted(registerList);
int registerIndex = -1;
do
diff --git a/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.h b/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.h
index 85b0b990dc407226cb12b4846dbeb4fea15d85fe..59c5bbb7f391ae2f4f1e7d589b01ec9f95335c02 100755
--- a/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.h
+++ b/silecs-diagnostic-cpp/src/silecs-diagnostic/silecsmodule.h
@@ -69,7 +69,7 @@ public:
/**
*Generate the empty entire tree
*/
- Item *generateTree(string className, string deployFile);
+ Item *generateTree(const std::string& className, const std::string& deployFile);
/**
* Set the device with the proper scalar data converting