minor refactoring
authorVictor Kirhenshtein <victor@netxms.org>
Tue, 7 Jul 2015 17:46:38 +0000 (20:46 +0300)
committerVictor Kirhenshtein <victor@netxms.org>
Tue, 7 Jul 2015 17:47:02 +0000 (20:47 +0300)
src/server/core/epp.cpp
src/server/core/node.cpp
src/server/core/objects.cpp
tests/test-libnetxms/nxcp.cpp

index c6691e0..3e2b5a9 100644 (file)
@@ -907,7 +907,7 @@ void EventPolicy::exportRule(String& str, const uuid& guid)
    readLock();
    for(UINT32 i = 0; i < m_dwNumRules; i++)
    {
-      if (!guid.compare(m_ppRuleList[i]->getGuid()))
+      if (guid.equals(m_ppRuleList[i]->getGuid()))
       {
          m_ppRuleList[i]->createNXMPRecord(str);
          break;
@@ -927,7 +927,7 @@ void EventPolicy::importRule(EPRule *rule)
    bool newRule = true;
    for(UINT32 i = 0; i < m_dwNumRules; i++)
    {
-      if (!rule->getGuid().compare(m_ppRuleList[i]->getGuid()))
+      if (rule->getGuid().equals(m_ppRuleList[i]->getGuid()))
       {
          delete m_ppRuleList[i];
          m_ppRuleList[i] = rule;
index 5b802c0..22b737f 100644 (file)
@@ -1705,7 +1705,7 @@ void Node::checkAgentPolicyBinding(AgentConnection *conn)
             int j;
                                for(j = 0; j < ap->size(); j++)
                                {
-                                       if (!ap->getGuid(j).compare(guid))
+                                       if (ap->getGuid(j).equals(guid))
                                                break;
                                }
                                if (j == ap->size())
index afe7ac2..583c716 100644 (file)
@@ -935,7 +935,7 @@ NetObj NXCORE_EXPORTABLE *FindObjectByName(const TCHAR *name, int objClass)
  */
 static bool ObjectGuidComparator(NetObj *object, void *data)
 {
-   return !object->isDeleted() && !object->getGuid().compare(*((const uuid *)data));
+   return !object->isDeleted() && object->getGuid().equals(*((const uuid *)data));
 }
 
 /**
index 2ec9d42..4e04f4e 100644 (file)
@@ -49,7 +49,7 @@ void TestMessageClass()
    uuid guid = uuid::generate();
    msg.setField(1, guid);
    uuid guid2 = msg.getFieldAsGUID(1);
-   AssertTrue(guid.compare(guid2) == 0);
+   AssertTrue(guid.equals(guid2));
 
    msg.setField(1, (UINT16)1234);
    AssertTrue(msg.getFieldAsUInt16(1) == 1234);
@@ -60,5 +60,9 @@ void TestMessageClass()
    msg.setField(1, (UINT64)1234);
    AssertTrue(msg.getFieldAsUInt64(1) == 1234);
 
+   msg.setField(1, _T("test text"));
+   TCHAR buffer[64];
+   AssertTrue(!_tcscmp(msg.getFieldAsString(1, buffer, 64), _T("test text")));
+
    EndTest();
 }