Skip to content

Commit

Permalink
added a missing header file and fixed incorrect format introduced in o…
Browse files Browse the repository at this point in the history
  • Loading branch information
JasonRuonanWang committed Feb 8, 2021
1 parent f891832 commit 5d1f5f2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
13 changes: 6 additions & 7 deletions source/adios2/core/Group.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -183,8 +183,7 @@ std::vector<std::string> Group::AvailableAttributes()
std::string variablePath = currentPath + groupDelimiter + v;
variablePath = variablePath.substr(
ADIOS_root.size() + 1, variablePath.size() - ADIOS_root.size());
if (attributes.find(variablePath) !=
attributes.end())
if (attributes.find(variablePath) != attributes.end())
{
available_attributes.push_back(v);
}
Expand All @@ -201,11 +200,11 @@ std::vector<std::string> Group::AvailableGroups()
std::set<std::string> val = mapPtr->treeMap[currentPath];
{
for (auto v : val)
{
if (mapPtr->treeMap.find(currentPath + groupDelimiter + v) !=
mapPtr->treeMap.end())
available_groups.push_back(v);
}
{
if (mapPtr->treeMap.find(currentPath + groupDelimiter + v) !=
mapPtr->treeMap.end())
available_groups.push_back(v);
}
}
return available_groups;
}
Expand Down
1 change: 1 addition & 0 deletions testing/adios2/hierarchy/TestHierarchicalReading.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
* Distributed under the OSI-approved Apache License, Version 2.0. See
* accompanying file Copyright.txt for details.
*/
#include <array>

This comment has been minimized.

Copy link
@dmitry-ganyushin

dmitry-ganyushin Feb 8, 2021

This test does not require . This work was not finished, I did not merge it manually

This comment has been minimized.

Copy link
@JasonRuonanWang

JasonRuonanWang via email Feb 8, 2021

Author Owner

This comment has been minimized.

Copy link
@JasonRuonanWang

JasonRuonanWang Feb 8, 2021

Author Owner

This test does not require . This work was not finished, I did not merge it manually

If you take a look at the current master branch, you will find that this test is already there. My suspicion is that you somehow accidentally pushed your commit directly into the master branch. But I am not sure.

#include <cstdint>
#include <cstring>

Expand Down

0 comments on commit 5d1f5f2

Please sign in to comment.