Skip to content

Commit dd382a3

Browse files
fixed a huge bug
1 parent a5847ae commit dd382a3

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

include/nbl/asset/utils/CCPUMeshPackerV2.h

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -100,10 +100,8 @@ uint32_t CCPUMeshPackerV2<MDIStructType>::commit(IMeshPackerBase::PackedMeshBuff
100100
size_t batchFirstIdx = ramb.indexAllocationOffset;
101101
size_t verticesAddedCnt = 0u;
102102
size_t instancesAddedCnt = 0u;
103-
uint32_t batchesAddedCnt = 0u;
104103

105104
const uint32_t batchCnt = triangleBatches.ranges.size() - 1u;
106-
batchCntTotal += batchCnt;
107105
for (uint32_t i = 0u; i < batchCnt; i++)
108106
{
109107
auto batchBegin = triangleBatches.ranges[i];
@@ -176,10 +174,10 @@ uint32_t CCPUMeshPackerV2<MDIStructType>::commit(IMeshPackerBase::PackedMeshBuff
176174
batchFirstIdx += idxInBatchCnt;
177175
}
178176

179-
instancesAddedCnt += insCnt;
177+
instancesAddedCnt += insCnt; // TODO: this a local variable... bug!
180178

181-
pmbd = { rambIn->mdiAllocationOffset + batchesAddedCnt, static_cast<uint32_t>(batchCnt) };
182-
batchesAddedCnt += batchCnt;
179+
pmbd = { rambIn->mdiAllocationOffset+batchCntTotal, static_cast<uint32_t>(batchCnt) };
180+
batchCntTotal += batchCnt;
183181

184182
i++;
185183
}

0 commit comments

Comments
 (0)