Skip to content

Commit 4858cef

Browse files
committed
DPL Analysis: Initialise ColumnIterator members
1 parent 3528ed2 commit 4858cef

1 file changed

Lines changed: 8 additions & 14 deletions

File tree

  • Framework/Core/include/Framework

‎Framework/Core/include/Framework/ASoA.h‎

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -506,13 +506,7 @@ class ColumnIterator : ChunkingPolicy
506506
/// it. This means that a ColumnIterator is actually only available
507507
/// as part of a RowView.
508508
ColumnIterator(arrow::ChunkedArray const* column)
509-
: mColumn{column},
510-
mCurrent{nullptr},
511-
mCurrentPos{nullptr},
512-
mGlobalOffset{nullptr},
513-
mLast{nullptr},
514-
mFirstIndex{0},
515-
mCurrentChunk{0}
509+
: mColumn{column}
516510
{
517511
auto array = getCurrentArray();
518512
mCurrent = reinterpret_cast<unwrap_t<T> const*>(array->values()->data());
@@ -609,13 +603,13 @@ class ColumnIterator : ChunkingPolicy
609603
return *this;
610604
}
611605

612-
mutable unwrap_t<T> const* mCurrent;
613-
int64_t const* mCurrentPos;
614-
uint64_t const* mGlobalOffset;
615-
mutable unwrap_t<T> const* mLast;
616-
arrow::ChunkedArray const* mColumn;
617-
mutable int mFirstIndex;
618-
mutable int mCurrentChunk;
606+
mutable unwrap_t<T> const* mCurrent{nullptr};
607+
int64_t const* mCurrentPos{nullptr};
608+
uint64_t const* mGlobalOffset{nullptr};
609+
mutable unwrap_t<T> const* mLast{nullptr};
610+
arrow::ChunkedArray const* mColumn{nullptr};
611+
mutable int mFirstIndex{0};
612+
mutable int mCurrentChunk{0};
619613

620614
private:
621615
void checkSkipChunk() const

0 commit comments

Comments
 (0)