Skip to content

Commit 06d7aad

Browse files
dfa1claude
andcommitted
refactor(reader): singleColumn drops its always-zero parameter
Found by IntelliJ inspections (value of colIdx is always 0) — dead generality from the typed-columns refactor. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent ce8fe9d commit 06d7aad

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

reader/src/main/java/io/github/dfa1/vortex/reader/ScanIterator.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -614,7 +614,7 @@ private SequencedMap<ColumnName, Chunk.Column> buildColumnMap(ChunkSpec chunk, A
614614
if (arr instanceof StructArray sa) {
615615
return expandStruct(sa);
616616
}
617-
return singleColumn(0, arr);
617+
return singleColumn(arr);
618618
}
619619
var scratch = new LinkedHashMap<ColumnName, Chunk.Column>(n);
620620
for (int i = 0; i < n; i++) {
@@ -637,7 +637,7 @@ private SequencedMap<ColumnName, Chunk.Column> buildSelfContainedColumnMap(Chunk
637637
if (arr instanceof StructArray sa) {
638638
return expandStruct(sa);
639639
}
640-
return singleColumn(0, arr);
640+
return singleColumn(arr);
641641
}
642642
var scratch = new LinkedHashMap<ColumnName, Chunk.Column>(n);
643643
for (int i = 0; i < n; i++) {
@@ -648,9 +648,9 @@ private SequencedMap<ColumnName, Chunk.Column> buildSelfContainedColumnMap(Chunk
648648
return unmodifiable(scratch);
649649
}
650650

651-
private SequencedMap<ColumnName, Chunk.Column> singleColumn(int colIdx, Array array) {
651+
private SequencedMap<ColumnName, Chunk.Column> singleColumn(Array array) {
652652
var map = new LinkedHashMap<ColumnName, Chunk.Column>(1);
653-
map.put(projectedNames.get(colIdx), new Chunk.Column(array, projectedDtypes.get(colIdx)));
653+
map.put(projectedNames.getFirst(), new Chunk.Column(array, projectedDtypes.getFirst()));
654654
return unmodifiable(map);
655655
}
656656

0 commit comments

Comments
 (0)