Skip to content

Commit d991fbe

Browse files
committed
fix undefined array return in Dimensions
1 parent ba20d25 commit d991fbe

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

components/omega/src/infra/Dimension.cpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
#include "Dimension.h"
1616
#include "DataTypes.h"
17+
#include "Error.h"
1718
#include "Logging.h"
1819
#include "OmegaKokkos.h"
1920
#include <map>
@@ -246,16 +247,15 @@ HostArray1DI4
246247
Dimension::getDimOffset(const std::string &Name // [in] name of dimension
247248
) {
248249

249-
// Make sure dimension exists
250-
if (exists(Name)) {
251-
std::shared_ptr<Dimension> ThisDim = AllDims[Name];
252-
return ThisDim->Offset;
250+
// Abort if the dimension does not exist
251+
if (!exists(Name))
252+
ABORT_ERROR("Cannot get offset array for dimension {}: "
253+
"dimension does not exist or has not been defined",
254+
Name);
253255

254-
} else {
255-
LOG_ERROR("Cannot get offset array for dimension {}: "
256-
"dimension does not exist or has not been defined",
257-
Name);
258-
}
256+
// Retrieve offset
257+
std::shared_ptr<Dimension> ThisDim = AllDims[Name];
258+
return ThisDim->Offset;
259259

260260
} // end getDimOffset
261261

0 commit comments

Comments
 (0)