Introduce 'RangeSequenceStorage'.

Author: fangerer

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/range/PRange.java

@@ -25,13 +25,13 @@
  */
 package com.oracle.graal.python.builtins.objects.range;
 
-import static com.oracle.graal.python.runtime.exception.PythonErrorType.IndexError;
 import static com.oracle.graal.python.runtime.exception.PythonErrorType.OverflowError;
 import static com.oracle.graal.python.runtime.exception.PythonErrorType.ValueError;
 
 import com.oracle.graal.python.PythonLanguage;
 import com.oracle.graal.python.builtins.objects.type.PythonClass;
 import com.oracle.graal.python.runtime.sequence.PImmutableSequence;
+import com.oracle.graal.python.runtime.sequence.storage.RangeSequenceStorage;
 import com.oracle.graal.python.runtime.sequence.storage.SequenceStorage;
 import com.oracle.truffle.api.CompilerAsserts;
 import com.oracle.truffle.api.CompilerDirectives;
@@ -100,12 +100,8 @@ public int getStop() {
         return stop;
     }
 
-    public Object getItemNormalized(int index) {
-        if (index >= length) {
-            CompilerDirectives.transferToInterpreter();
-            throw PythonLanguage.getCore().raise(IndexError, "range object index out of range");
-        }
-
+    public int getItemNormalized(int index) {
+        assert index < length;
         return index * step + start;
     }
 
@@ -116,8 +112,7 @@ public int len() {
 
     @Override
     public SequenceStorage getSequenceStorage() {
-        CompilerDirectives.transferToInterpreter();
-        throw new UnsupportedOperationException();
+        return new RangeSequenceStorage(this);
     }
 
     @Override
@@ -145,5 +140,4 @@ public boolean equals(Object other) {
             return false;
         }
     }
-
 }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/sequence/storage/RangeSequenceStorage.java

@@ -0,0 +1,154 @@
+package com.oracle.graal.python.runtime.sequence.storage;
+
+import com.oracle.graal.python.builtins.objects.range.PRange;
+
+public class RangeSequenceStorage extends SequenceStorage {
+
+    private final PRange range;
+
+    public RangeSequenceStorage(PRange range) {
+        this.range = range;
+    }
+
+    public PRange getRange() {
+        return range;
+    }
+
+    @Override
+    public int length() {
+        return range.len();
+    }
+
+    @Override
+    public void setNewLength(int length) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public SequenceStorage copy() {
+        return this;
+    }
+
+    @Override
+    public SequenceStorage createEmpty(int newCapacity) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public Object getInternalArrayObject() {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public ListStorageType getElementType() {
+        return ListStorageType.Int;
+    }
+
+    @Override
+    public Object[] getInternalArray() {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public Object[] getCopyOfInternalArray() {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public Object getItemNormalized(int idx) {
+        return range.getItemNormalized(idx);
+    }
+
+    public int getIntItemNormalized(int idx) {
+        return range.getItemNormalized(idx);
+    }
+
+    @Override
+    public void setItemNormalized(int idx, Object value) throws SequenceStoreException {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void insertItem(int idx, Object value) throws SequenceStoreException {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public SequenceStorage getSliceInBound(int start, int stop, int step, int length) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void setSliceInBound(int start, int stop, int step, SequenceStorage sequence) throws SequenceStoreException {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void delSlice(int start, int stop, int step) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void delItemInBound(int idx) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public Object popInBound(int idx) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public int index(Object value) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void append(Object value) throws SequenceStoreException {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void extend(SequenceStorage other) throws SequenceStoreException, ArithmeticException {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void reverse() {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void sort() {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public boolean equals(SequenceStorage other) {
+        if (other instanceof RangeSequenceStorage) {
+            return range.equals(((RangeSequenceStorage) other).range);
+        }
+        return false;
+    }
+
+    @Override
+    public SequenceStorage generalizeFor(Object value, SequenceStorage other) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public Object getIndicativeValue() {
+        return range.getStart();
+    }
+
+    @Override
+    public void ensureCapacity(int newCapacity) {
+        throw new UnsupportedOperationException();
+    }
+
+    @Override
+    public void copyItem(int idxTo, int idxFrom) {
+        throw new UnsupportedOperationException();
+    }
+
+}