Changeset 908

Timestamp:

07/28/16 15:43:07 (8 years ago)

Author:

rlacroix

Message:

Fix: The buffers sizes for servers to clients communications were overestimated.

The servers do not need to send the attributes to the clients so there is no need to account for those when computing the buffers sizes.

File:

• XIOS/trunk/src/node/context.cpp (modified) (1 diff)

XIOS/trunk/src/node/context.cpp

@@ -837 +837 @@
      std::map<int, StdSize> attributesSize;
 
-     size_t numEnabledFiles = this->enabledFiles.size();
-     for (size_t i = 0; i < numEnabledFiles; ++i)
-     {
-       CFile* file = this->enabledFiles[i];
-
-       std::vector<CField*> enabledFields = file->getEnabledFields();
-       size_t numEnabledFields = enabledFields.size();
-       for (size_t j = 0; j < numEnabledFields; ++j)
+     if (hasClient)
+     {
+       size_t numEnabledFiles = this->enabledFiles.size();
+       for (size_t i = 0; i < numEnabledFiles; ++i)
        {
-         const std::map<int, StdSize> mapSize = enabledFields[j]->getGridAttributesBufferSize();
-         std::map<int, StdSize>::const_iterator it = mapSize.begin(), itE = mapSize.end();
-         for (; it != itE; ++it)
+         CFile* file = this->enabledFiles[i];
+
+         std::vector<CField*> enabledFields = file->getEnabledFields();
+         size_t numEnabledFields = enabledFields.size();
+         for (size_t j = 0; j < numEnabledFields; ++j)
          {
-           // If attributesSize[it->first] does not exist, it will be zero-initialized
-           // so we can use it safely without checking for its existance
-           if (attributesSize[it->first] < it->second)
-             attributesSize[it->first] = it->second;
+           const std::map<int, StdSize> mapSize = enabledFields[j]->getGridAttributesBufferSize();
+           std::map<int, StdSize>::const_iterator it = mapSize.begin(), itE = mapSize.end();
+           for (; it != itE; ++it)
+           {
+             // If attributesSize[it->first] does not exist, it will be zero-initialized
+             // so we can use it safely without checking for its existance
+             if (attributesSize[it->first] < it->second)
+               attributesSize[it->first] = it->second;
+           }
          }
        }