Skip to content

Commit e6b31d4

Browse files
committed
Fix: did not use provided index
1 parent b73ec97 commit e6b31d4

File tree

1 file changed

+5
-5
lines changed
  • graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/hpy

1 file changed

+5
-5
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/hpy/GraalHPyContext.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1413,10 +1413,10 @@ public final long ctxGetItemi(long handle, long lidx) {
14131413
Object receiver = getObjectForHPyHandle(GraalHPyBoxing.unboxHandle(handle)).getDelegate();
14141414
Object clazz = GetClassNode.getUncached().execute(receiver);
14151415
if (clazz == PythonBuiltinClassType.PList || clazz == PythonBuiltinClassType.PTuple) {
1416-
int idx = 0;
1417-
if (!com.oracle.graal.python.builtins.objects.ints.PInt.isIntRange(lidx)) {
1416+
if (!PInt.isIntRange(lidx)) {
14181417
throw PRaiseNode.raiseUncached(null, PythonBuiltinClassType.IndexError, ErrorMessages.CANNOT_FIT_P_INTO_INDEXSIZED_INT, lidx);
14191418
}
1419+
int idx = (int) lidx;
14201420
PSequence sequence = (PSequence) receiver;
14211421
SequenceStorage storage = sequence.getSequenceStorage();
14221422
if (storage instanceof IntSequenceStorage) {
@@ -1519,11 +1519,11 @@ public final int ctxSetItemi(long hSequence, long lidx, long hValue) {
15191519

15201520
private boolean ctxListSetItem(Object receiver, long lidx, long hValue) {
15211521
// fast path for list
1522-
int idx = 0;
1523-
if (!com.oracle.graal.python.builtins.objects.ints.PInt.isIntRange(lidx)) {
1522+
if (!PInt.isIntRange(lidx)) {
15241523
throw PRaiseNode.raiseUncached(null, PythonBuiltinClassType.IndexError, ErrorMessages.CANNOT_FIT_P_INTO_INDEXSIZED_INT, lidx);
15251524
}
1526-
com.oracle.graal.python.builtins.objects.list.PList sequence = (PList) receiver;
1525+
int idx = (int) lidx;
1526+
PList sequence = (PList) receiver;
15271527
SequenceStorage storage = sequence.getSequenceStorage();
15281528
if (storage instanceof IntSequenceStorage && GraalHPyBoxing.isBoxedInt(hValue)) {
15291529
((IntSequenceStorage) storage).setIntItemNormalized(idx, GraalHPyBoxing.unboxInt(hValue));

0 commit comments

Comments
 (0)