Skip to content

Commit 4589d82

Browse files
committed
Fix Win32 Narrowing Error
Fix ``` 2025-01-16T05:51:21.6362891Z D:\a\openPMD-api\openPMD-api\src\dep-adios2\ADIOS2-2.10.2\source\adios2\toolkit\format\bp5\BP5Serializer.cpp(1603,68): error C2397: conversion from 'uint64_t' to 'size_t' requires a narrowing conversion [D:\a\openPMD-api\openPMD-api\src\build-adios2\source\adios2\adios2_core.vcxproj] 2025-01-16T05:51:21.6371102Z D:\a\openPMD-api\openPMD-api\src\dep-adios2\ADIOS2-2.10.2\source\adios2\toolkit\format\bp5\BP5Serializer.cpp(1611,69): error C2397: conversion from 'uint64_t' to 'size_t' requires a narrowing conversion [D:\a\openPMD-api\openPMD-api\src\build-adios2\source\adios2\adios2_core.vcxproj] ``` with MSVC on Win32 ILP32. `core::iovec` is a `void*` and `size_t`, so this part needs to be `size_t` as well.
1 parent a19dad6 commit 4589d82

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

source/adios2/toolkit/format/bp5/BP5Serializer.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1596,17 +1596,17 @@ std::vector<core::iovec> BP5Serializer::BreakoutContiguousMetadata(
15961596
}
15971597
}
15981598
helper::CopyFromBuffer(Aggregate.data(), Position, &MBCount);
1599-
for (uint64_t i = 0; i < MBCount; ++i)
1599+
for (size_t i = 0; i < MBCount; ++i)
16001600
{
1601-
uint64_t MEBSize;
1601+
size_t MEBSize;
16021602
helper::CopyFromBuffer(Aggregate.data(), Position, &MEBSize);
16031603
MetadataBlocks.push_back({Aggregate.data() + Position, MEBSize});
16041604
Position += MEBSize;
16051605
}
16061606
helper::CopyFromBuffer(Aggregate.data(), Position, &ABCount);
1607-
for (uint64_t i = 0; i < ABCount; ++i)
1607+
for (size_t i = 0; i < ABCount; ++i)
16081608
{
1609-
uint64_t AEBSize;
1609+
size_t AEBSize;
16101610
helper::CopyFromBuffer(Aggregate.data(), Position, &AEBSize);
16111611
AttributeBlocks.push_back({Aggregate.data() + Position, AEBSize});
16121612
Position += AEBSize;

0 commit comments

Comments
 (0)