Skip to content

Commit f120234

Browse files
committed
Use setStoreAndSize more widely to ensure assertion are run in CI.
1 parent 2d5d7cf commit f120234

File tree

1 file changed

+7
-14
lines changed

1 file changed

+7
-14
lines changed

src/main/java/org/truffleruby/core/array/ArrayNodes.java

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -577,8 +577,7 @@ protected Object compactObjects(RubyArray array,
577577

578578
stores.clear(oldStore, m, size - m);
579579

580-
array.store = newStore;
581-
array.size = m;
580+
setStoreAndSize(array, newStore, m);
582581

583582
if (m == size) {
584583
return nil;
@@ -740,8 +739,7 @@ private Object delete(RubyArray array, Object value, Object maybeBlock,
740739
if (sameStores) {
741740
oldStores.clear(oldStore, i, size - i);
742741
}
743-
array.store = newStore;
744-
array.size = i;
742+
setStoreAndSize(array, newStore, i);
745743
return found;
746744
} else {
747745
if (maybeBlock == nil) {
@@ -794,8 +792,7 @@ protected Object deleteAt(RubyArray array, int index,
794792
final Object value = stores.read(store, i);
795793
stores.copyContents(store, i + 1, store, i, size - i - 1);
796794
stores.clear(store, size - 1, 1);
797-
array.store = store;
798-
array.size = size - 1;
795+
setStoreAndSize(array, store, size - 1);
799796
return value;
800797
}
801798
}
@@ -822,8 +819,7 @@ protected Object deleteAtCopying(RubyArray array, int index,
822819
stores.copyContents(store, 0, mutableStore, 0, i);
823820
final Object value = stores.read(store, i);
824821
stores.copyContents(store, i + 1, mutableStore, i, size - i - 1);
825-
array.store = mutableStore;
826-
array.size = size - 1;
822+
setStoreAndSize(array, mutableStore, size - 1);
827823
return value;
828824
}
829825
}
@@ -2141,8 +2137,7 @@ protected Object shiftOther(RubyArray array, NotProvided n,
21412137
final int size = array.size;
21422138
final Object value = stores.read(store, 0);
21432139
stores.clear(store, 0, 1);
2144-
array.store = stores.extractRange(store, 1, size);
2145-
setSize(array, size - 1);
2140+
setStoreAndSize(array, stores.extractRange(store, 1, size), size - 1);
21462141
return value;
21472142
}
21482143

@@ -2310,10 +2305,8 @@ protected RubyArray stealStorage(RubyArray array, RubyArray other,
23102305

23112306
final int size = other.size;
23122307
final Object store = other.store;
2313-
array.store = store;
2314-
array.size = size;
2315-
other.store = stores.initialStore(store);
2316-
other.size = 0;
2308+
setStoreAndSize(array, store, size);
2309+
setStoreAndSize(other, stores.initialStore(store), 0);
23172310

23182311
return array;
23192312
}

0 commit comments

Comments
 (0)