Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 1 | /* Range object implementation */ |
| 2 | |
Guido van Rossum | c0b618a | 1997-05-02 03:12:38 +0000 | [diff] [blame] | 3 | #include "Python.h" |
Thomas Wouters | efafcea | 2001-07-09 12:30:54 +0000 | [diff] [blame] | 4 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 5 | /* Support objects whose length is > PY_SSIZE_T_MAX. |
| 6 | |
| 7 | This could be sped up for small PyLongs if they fit in an Py_ssize_t. |
| 8 | This only matters on Win64. Though we could use PY_LONG_LONG which |
| 9 | would presumably help perf. |
| 10 | */ |
| 11 | |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 12 | typedef struct { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 13 | PyObject_HEAD |
| 14 | PyObject *start; |
| 15 | PyObject *stop; |
| 16 | PyObject *step; |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 17 | } rangeobject; |
| 18 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 19 | /* Helper function for validating step. Always returns a new reference or |
| 20 | NULL on error. |
| 21 | */ |
| 22 | static PyObject * |
| 23 | validate_step(PyObject *step) |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 24 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 25 | /* No step specified, use a step of 1. */ |
| 26 | if (!step) |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 27 | return PyLong_FromLong(1); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 28 | |
| 29 | step = PyNumber_Index(step); |
| 30 | if (step) { |
| 31 | Py_ssize_t istep = PyNumber_AsSsize_t(step, NULL); |
| 32 | if (istep == -1 && PyErr_Occurred()) { |
| 33 | /* Ignore OverflowError, we know the value isn't 0. */ |
| 34 | PyErr_Clear(); |
| 35 | } |
| 36 | else if (istep == 0) { |
| 37 | PyErr_SetString(PyExc_ValueError, |
| 38 | "range() arg 3 must not be zero"); |
| 39 | Py_CLEAR(step); |
| 40 | } |
| 41 | } |
| 42 | |
| 43 | return step; |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 44 | } |
| 45 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 46 | /* XXX(nnorwitz): should we error check if the user passes any empty ranges? |
| 47 | range(-10) |
| 48 | range(0, -5) |
| 49 | range(0, 5, -1) |
| 50 | */ |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 51 | static PyObject * |
| 52 | range_new(PyTypeObject *type, PyObject *args, PyObject *kw) |
| 53 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 54 | rangeobject *obj = NULL; |
| 55 | PyObject *start = NULL, *stop = NULL, *step = NULL; |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 56 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 57 | if (!_PyArg_NoKeywords("range()", kw)) |
| 58 | return NULL; |
Georg Brandl | 02c4287 | 2005-08-26 06:42:30 +0000 | [diff] [blame] | 59 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 60 | if (PyTuple_Size(args) <= 1) { |
| 61 | if (!PyArg_UnpackTuple(args, "range", 1, 1, &stop)) |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 62 | return NULL; |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 63 | stop = PyNumber_Index(stop); |
| 64 | if (!stop) |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 65 | return NULL; |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 66 | start = PyLong_FromLong(0); |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 67 | if (!start) { |
| 68 | Py_DECREF(stop); |
| 69 | return NULL; |
| 70 | } |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 71 | step = PyLong_FromLong(1); |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 72 | if (!step) { |
| 73 | Py_DECREF(stop); |
| 74 | Py_DECREF(start); |
| 75 | return NULL; |
| 76 | } |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 77 | } |
| 78 | else { |
| 79 | if (!PyArg_UnpackTuple(args, "range", 2, 3, |
| 80 | &start, &stop, &step)) |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 81 | return NULL; |
Tim Peters | feec453 | 2004-08-08 07:17:39 +0000 | [diff] [blame] | 82 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 83 | /* Convert borrowed refs to owned refs */ |
| 84 | start = PyNumber_Index(start); |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 85 | if (!start) |
| 86 | return NULL; |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 87 | stop = PyNumber_Index(stop); |
Raymond Hettinger | 94f5583 | 2009-06-12 18:40:16 +0000 | [diff] [blame] | 88 | if (!stop) { |
| 89 | Py_DECREF(start); |
| 90 | return NULL; |
| 91 | } |
| 92 | step = validate_step(step); /* Caution, this can clear exceptions */ |
| 93 | if (!step) { |
| 94 | Py_DECREF(start); |
| 95 | Py_DECREF(stop); |
| 96 | return NULL; |
| 97 | } |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 98 | } |
| 99 | |
| 100 | obj = PyObject_New(rangeobject, &PyRange_Type); |
| 101 | if (obj == NULL) |
| 102 | goto Fail; |
| 103 | obj->start = start; |
| 104 | obj->stop = stop; |
| 105 | obj->step = step; |
| 106 | return (PyObject *) obj; |
| 107 | |
| 108 | Fail: |
| 109 | Py_XDECREF(start); |
| 110 | Py_XDECREF(stop); |
| 111 | Py_XDECREF(step); |
| 112 | return NULL; |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 113 | } |
| 114 | |
Martin v. Löwis | 14f8b4c | 2002-06-13 20:33:02 +0000 | [diff] [blame] | 115 | PyDoc_STRVAR(range_doc, |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 116 | "range([start,] stop[, step]) -> range object\n\ |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 117 | \n\ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 118 | Returns an iterator that generates the numbers in the range on demand."); |
Raymond Hettinger | c4c453f | 2002-06-05 23:12:45 +0000 | [diff] [blame] | 119 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 120 | static void |
| 121 | range_dealloc(rangeobject *r) |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 122 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 123 | Py_DECREF(r->start); |
| 124 | Py_DECREF(r->stop); |
| 125 | Py_DECREF(r->step); |
Amaury Forgeot d'Arc | b7f17e4 | 2007-11-15 20:52:21 +0000 | [diff] [blame] | 126 | PyObject_Del(r); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 127 | } |
| 128 | |
| 129 | /* Return number of items in range (lo, hi, step), when arguments are |
| 130 | * PyInt or PyLong objects. step > 0 required. Return a value < 0 if |
| 131 | * & only if the true value is too large to fit in a signed long. |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 132 | * Arguments MUST return 1 with either PyLong_Check() or |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 133 | * PyLong_Check(). Return -1 when there is an error. |
| 134 | */ |
| 135 | static PyObject* |
| 136 | range_length_obj(rangeobject *r) |
| 137 | { |
| 138 | /* ------------------------------------------------------------- |
| 139 | Algorithm is equal to that of get_len_of_range(), but it operates |
| 140 | on PyObjects (which are assumed to be PyLong or PyInt objects). |
| 141 | ---------------------------------------------------------------*/ |
Mark Dickinson | 211c625 | 2009-02-01 10:28:51 +0000 | [diff] [blame] | 142 | int cmp_result; |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 143 | PyObject *lo, *hi; |
| 144 | PyObject *step = NULL; |
| 145 | PyObject *diff = NULL; |
| 146 | PyObject *one = NULL; |
| 147 | PyObject *tmp1 = NULL, *tmp2 = NULL, *result; |
| 148 | /* holds sub-expression evaluations */ |
| 149 | |
| 150 | PyObject *zero = PyLong_FromLong(0); |
| 151 | if (zero == NULL) |
| 152 | return NULL; |
Mark Dickinson | 211c625 | 2009-02-01 10:28:51 +0000 | [diff] [blame] | 153 | cmp_result = PyObject_RichCompareBool(r->step, zero, Py_GT); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 154 | Py_DECREF(zero); |
Mark Dickinson | 211c625 | 2009-02-01 10:28:51 +0000 | [diff] [blame] | 155 | if (cmp_result == -1) |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 156 | return NULL; |
| 157 | |
Mark Dickinson | 211c625 | 2009-02-01 10:28:51 +0000 | [diff] [blame] | 158 | if (cmp_result == 1) { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 159 | lo = r->start; |
| 160 | hi = r->stop; |
| 161 | step = r->step; |
| 162 | Py_INCREF(step); |
| 163 | } else { |
| 164 | lo = r->stop; |
| 165 | hi = r->start; |
| 166 | step = PyNumber_Negative(r->step); |
| 167 | if (!step) |
| 168 | return NULL; |
| 169 | } |
| 170 | |
| 171 | /* if (lo >= hi), return length of 0. */ |
Mark Dickinson | 211c625 | 2009-02-01 10:28:51 +0000 | [diff] [blame] | 172 | if (PyObject_RichCompareBool(lo, hi, Py_GE) == 1) { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 173 | Py_XDECREF(step); |
| 174 | return PyLong_FromLong(0); |
| 175 | } |
| 176 | |
| 177 | if ((one = PyLong_FromLong(1L)) == NULL) |
| 178 | goto Fail; |
| 179 | |
| 180 | if ((tmp1 = PyNumber_Subtract(hi, lo)) == NULL) |
| 181 | goto Fail; |
| 182 | |
| 183 | if ((diff = PyNumber_Subtract(tmp1, one)) == NULL) |
| 184 | goto Fail; |
| 185 | |
| 186 | if ((tmp2 = PyNumber_FloorDivide(diff, step)) == NULL) |
| 187 | goto Fail; |
| 188 | |
| 189 | if ((result = PyNumber_Add(tmp2, one)) == NULL) |
| 190 | goto Fail; |
| 191 | |
| 192 | Py_DECREF(tmp2); |
| 193 | Py_DECREF(diff); |
| 194 | Py_DECREF(step); |
| 195 | Py_DECREF(tmp1); |
| 196 | Py_DECREF(one); |
| 197 | return result; |
| 198 | |
| 199 | Fail: |
| 200 | Py_XDECREF(tmp2); |
| 201 | Py_XDECREF(diff); |
| 202 | Py_XDECREF(step); |
| 203 | Py_XDECREF(tmp1); |
| 204 | Py_XDECREF(one); |
| 205 | return NULL; |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 206 | } |
| 207 | |
Martin v. Löwis | 18e1655 | 2006-02-15 17:27:45 +0000 | [diff] [blame] | 208 | static Py_ssize_t |
Fred Drake | 45cfbcc | 2000-07-09 06:21:27 +0000 | [diff] [blame] | 209 | range_length(rangeobject *r) |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 210 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 211 | PyObject *len = range_length_obj(r); |
| 212 | Py_ssize_t result = -1; |
| 213 | if (len) { |
| 214 | result = PyLong_AsSsize_t(len); |
| 215 | Py_DECREF(len); |
| 216 | } |
| 217 | return result; |
| 218 | } |
| 219 | |
| 220 | /* range(...)[x] is necessary for: seq[:] = range(...) */ |
| 221 | |
| 222 | static PyObject * |
| 223 | range_item(rangeobject *r, Py_ssize_t i) |
| 224 | { |
| 225 | Py_ssize_t len = range_length(r); |
| 226 | PyObject *rem, *incr, *result; |
| 227 | |
| 228 | /* XXX(nnorwitz): should negative indices be supported? */ |
| 229 | /* XXX(nnorwitz): should support range[x] where x > PY_SSIZE_T_MAX? */ |
| 230 | if (i < 0 || i >= len) { |
| 231 | if (!PyErr_Occurred()) |
| 232 | PyErr_SetString(PyExc_IndexError, |
Walter Dörwald | 4ad9421 | 2007-05-21 18:01:17 +0000 | [diff] [blame] | 233 | "range object index out of range"); |
Benjamin Peterson | df0a5cb | 2008-04-25 21:15:37 +0000 | [diff] [blame] | 234 | return NULL; |
| 235 | } |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 236 | |
| 237 | /* XXX(nnorwitz): optimize for short ints. */ |
Raymond Hettinger | ad3f332 | 2008-02-09 04:13:49 +0000 | [diff] [blame] | 238 | rem = PyLong_FromSsize_t(i); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 239 | if (!rem) |
| 240 | return NULL; |
| 241 | incr = PyNumber_Multiply(rem, r->step); |
| 242 | Py_DECREF(rem); |
| 243 | if (!incr) |
| 244 | return NULL; |
| 245 | result = PyNumber_Add(r->start, incr); |
| 246 | Py_DECREF(incr); |
| 247 | return result; |
Guido van Rossum | 7d6aa51 | 1993-12-21 22:50:31 +0000 | [diff] [blame] | 248 | } |
| 249 | |
Guido van Rossum | c0b618a | 1997-05-02 03:12:38 +0000 | [diff] [blame] | 250 | static PyObject * |
Fred Drake | 45cfbcc | 2000-07-09 06:21:27 +0000 | [diff] [blame] | 251 | range_repr(rangeobject *r) |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 252 | { |
Walter Dörwald | 03b43d8 | 2007-05-21 10:43:34 +0000 | [diff] [blame] | 253 | Py_ssize_t istep; |
Tim Peters | d976ab7 | 2004-08-08 06:29:10 +0000 | [diff] [blame] | 254 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 255 | /* Check for special case values for printing. We don't always |
Walter Dörwald | 03b43d8 | 2007-05-21 10:43:34 +0000 | [diff] [blame] | 256 | need the step value. We don't care about errors |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 257 | (it means overflow), so clear the errors. */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 258 | istep = PyNumber_AsSsize_t(r->step, NULL); |
| 259 | if (istep != 1 || (istep == -1 && PyErr_Occurred())) { |
| 260 | PyErr_Clear(); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 261 | } |
| 262 | |
Walter Dörwald | 03b43d8 | 2007-05-21 10:43:34 +0000 | [diff] [blame] | 263 | if (istep == 1) |
Walter Dörwald | 850e516 | 2007-05-20 08:19:54 +0000 | [diff] [blame] | 264 | return PyUnicode_FromFormat("range(%R, %R)", r->start, r->stop); |
| 265 | else |
| 266 | return PyUnicode_FromFormat("range(%R, %R, %R)", |
| 267 | r->start, r->stop, r->step); |
Thomas Wouters | efafcea | 2001-07-09 12:30:54 +0000 | [diff] [blame] | 268 | } |
| 269 | |
Alexandre Vassalotti | 7505607 | 2008-06-10 04:03:04 +0000 | [diff] [blame] | 270 | /* Pickling support */ |
| 271 | static PyObject * |
| 272 | range_reduce(rangeobject *r, PyObject *args) |
| 273 | { |
| 274 | return Py_BuildValue("(O(OOO))", Py_TYPE(r), |
| 275 | r->start, r->stop, r->step); |
| 276 | } |
| 277 | |
Guido van Rossum | c0b618a | 1997-05-02 03:12:38 +0000 | [diff] [blame] | 278 | static PySequenceMethods range_as_sequence = { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 279 | (lenfunc)range_length, /* sq_length */ |
| 280 | 0, /* sq_concat */ |
| 281 | 0, /* sq_repeat */ |
| 282 | (ssizeargfunc)range_item, /* sq_item */ |
| 283 | 0, /* sq_slice */ |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 284 | }; |
| 285 | |
Jeremy Hylton | 938ace6 | 2002-07-17 16:30:39 +0000 | [diff] [blame] | 286 | static PyObject * range_iter(PyObject *seq); |
Raymond Hettinger | 85c20a4 | 2003-11-06 14:06:48 +0000 | [diff] [blame] | 287 | static PyObject * range_reverse(PyObject *seq); |
| 288 | |
| 289 | PyDoc_STRVAR(reverse_doc, |
| 290 | "Returns a reverse iterator."); |
| 291 | |
| 292 | static PyMethodDef range_methods[] = { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 293 | {"__reversed__", (PyCFunction)range_reverse, METH_NOARGS, |
| 294 | reverse_doc}, |
Alexandre Vassalotti | 7505607 | 2008-06-10 04:03:04 +0000 | [diff] [blame] | 295 | {"__reduce__", (PyCFunction)range_reduce, METH_VARARGS}, |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 296 | {NULL, NULL} /* sentinel */ |
Raymond Hettinger | 85c20a4 | 2003-11-06 14:06:48 +0000 | [diff] [blame] | 297 | }; |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 298 | |
Guido van Rossum | c0b618a | 1997-05-02 03:12:38 +0000 | [diff] [blame] | 299 | PyTypeObject PyRange_Type = { |
Martin v. Löwis | 9f2e346 | 2007-07-21 17:22:18 +0000 | [diff] [blame] | 300 | PyVarObject_HEAD_INIT(&PyType_Type, 0) |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 301 | "range", /* Name of this type */ |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 302 | sizeof(rangeobject), /* Basic object size */ |
| 303 | 0, /* Item size for varobject */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 304 | (destructor)range_dealloc, /* tp_dealloc */ |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 305 | 0, /* tp_print */ |
| 306 | 0, /* tp_getattr */ |
| 307 | 0, /* tp_setattr */ |
Mark Dickinson | e94c679 | 2009-02-02 20:36:42 +0000 | [diff] [blame] | 308 | 0, /* tp_reserved */ |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 309 | (reprfunc)range_repr, /* tp_repr */ |
| 310 | 0, /* tp_as_number */ |
| 311 | &range_as_sequence, /* tp_as_sequence */ |
| 312 | 0, /* tp_as_mapping */ |
| 313 | 0, /* tp_hash */ |
| 314 | 0, /* tp_call */ |
| 315 | 0, /* tp_str */ |
| 316 | PyObject_GenericGetAttr, /* tp_getattro */ |
| 317 | 0, /* tp_setattro */ |
| 318 | 0, /* tp_as_buffer */ |
| 319 | Py_TPFLAGS_DEFAULT, /* tp_flags */ |
| 320 | range_doc, /* tp_doc */ |
| 321 | 0, /* tp_traverse */ |
| 322 | 0, /* tp_clear */ |
| 323 | 0, /* tp_richcompare */ |
| 324 | 0, /* tp_weaklistoffset */ |
| 325 | range_iter, /* tp_iter */ |
| 326 | 0, /* tp_iternext */ |
| 327 | range_methods, /* tp_methods */ |
| 328 | 0, /* tp_members */ |
| 329 | 0, /* tp_getset */ |
| 330 | 0, /* tp_base */ |
| 331 | 0, /* tp_dict */ |
| 332 | 0, /* tp_descr_get */ |
| 333 | 0, /* tp_descr_set */ |
| 334 | 0, /* tp_dictoffset */ |
| 335 | 0, /* tp_init */ |
| 336 | 0, /* tp_alloc */ |
| 337 | range_new, /* tp_new */ |
Guido van Rossum | 12d12c5 | 1993-10-26 17:58:25 +0000 | [diff] [blame] | 338 | }; |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 339 | |
Walter Dörwald | 4ad9421 | 2007-05-21 18:01:17 +0000 | [diff] [blame] | 340 | /*********************** range Iterator **************************/ |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 341 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 342 | /* There are 2 types of iterators, one for C longs, the other for |
| 343 | Python longs (ie, PyObjects). This should make iteration fast |
| 344 | in the normal case, but possible for any numeric value. |
| 345 | */ |
| 346 | |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 347 | typedef struct { |
| 348 | PyObject_HEAD |
| 349 | long index; |
| 350 | long start; |
| 351 | long step; |
| 352 | long len; |
| 353 | } rangeiterobject; |
| 354 | |
Raymond Hettinger | 85c20a4 | 2003-11-06 14:06:48 +0000 | [diff] [blame] | 355 | static PyObject * |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 356 | rangeiter_next(rangeiterobject *r) |
| 357 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 358 | if (r->index < r->len) |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 359 | return PyLong_FromLong(r->start + (r->index++) * r->step); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 360 | return NULL; |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 361 | } |
| 362 | |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 363 | static PyObject * |
Raymond Hettinger | ef9bf40 | 2004-03-10 10:10:42 +0000 | [diff] [blame] | 364 | rangeiter_len(rangeiterobject *r) |
| 365 | { |
Christian Heimes | 217cfd1 | 2007-12-02 14:31:20 +0000 | [diff] [blame] | 366 | return PyLong_FromLong(r->len - r->index); |
Raymond Hettinger | ef9bf40 | 2004-03-10 10:10:42 +0000 | [diff] [blame] | 367 | } |
| 368 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 369 | typedef struct { |
| 370 | PyObject_HEAD |
| 371 | PyObject *index; |
| 372 | PyObject *start; |
| 373 | PyObject *step; |
| 374 | PyObject *len; |
| 375 | } longrangeiterobject; |
| 376 | |
| 377 | static PyObject * |
| 378 | longrangeiter_len(longrangeiterobject *r, PyObject *no_args) |
| 379 | { |
| 380 | return PyNumber_Subtract(r->len, r->index); |
| 381 | } |
Guido van Rossum | 317e774 | 2007-05-08 15:18:31 +0000 | [diff] [blame] | 382 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 383 | static PyObject *rangeiter_new(PyTypeObject *, PyObject *args, PyObject *kw); |
| 384 | |
| 385 | PyDoc_STRVAR(length_hint_doc, |
| 386 | "Private method returning an estimate of len(list(it))."); |
Raymond Hettinger | ef9bf40 | 2004-03-10 10:10:42 +0000 | [diff] [blame] | 387 | |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 388 | static PyMethodDef rangeiter_methods[] = { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 389 | {"__length_hint__", (PyCFunction)rangeiter_len, METH_NOARGS, |
| 390 | length_hint_doc}, |
| 391 | {NULL, NULL} /* sentinel */ |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 392 | }; |
Raymond Hettinger | ef9bf40 | 2004-03-10 10:10:42 +0000 | [diff] [blame] | 393 | |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 394 | PyTypeObject PyRangeIter_Type = { |
Martin v. Löwis | 9f2e346 | 2007-07-21 17:22:18 +0000 | [diff] [blame] | 395 | PyVarObject_HEAD_INIT(&PyType_Type, 0) |
Christian Heimes | f83be4e | 2007-11-28 09:44:38 +0000 | [diff] [blame] | 396 | "range_iterator", /* tp_name */ |
Neal Norwitz | 56f46f8 | 2002-06-06 14:58:21 +0000 | [diff] [blame] | 397 | sizeof(rangeiterobject), /* tp_basicsize */ |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 398 | 0, /* tp_itemsize */ |
| 399 | /* methods */ |
| 400 | (destructor)PyObject_Del, /* tp_dealloc */ |
| 401 | 0, /* tp_print */ |
| 402 | 0, /* tp_getattr */ |
| 403 | 0, /* tp_setattr */ |
Mark Dickinson | e94c679 | 2009-02-02 20:36:42 +0000 | [diff] [blame] | 404 | 0, /* tp_reserved */ |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 405 | 0, /* tp_repr */ |
| 406 | 0, /* tp_as_number */ |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 407 | 0, /* tp_as_sequence */ |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 408 | 0, /* tp_as_mapping */ |
| 409 | 0, /* tp_hash */ |
| 410 | 0, /* tp_call */ |
| 411 | 0, /* tp_str */ |
| 412 | PyObject_GenericGetAttr, /* tp_getattro */ |
| 413 | 0, /* tp_setattro */ |
| 414 | 0, /* tp_as_buffer */ |
| 415 | Py_TPFLAGS_DEFAULT, /* tp_flags */ |
| 416 | 0, /* tp_doc */ |
| 417 | 0, /* tp_traverse */ |
| 418 | 0, /* tp_clear */ |
| 419 | 0, /* tp_richcompare */ |
| 420 | 0, /* tp_weaklistoffset */ |
Raymond Hettinger | 1da1dbf | 2003-03-17 19:46:11 +0000 | [diff] [blame] | 421 | PyObject_SelfIter, /* tp_iter */ |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 422 | (iternextfunc)rangeiter_next, /* tp_iternext */ |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 423 | rangeiter_methods, /* tp_methods */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 424 | 0, /* tp_members */ |
| 425 | 0, /* tp_getset */ |
| 426 | 0, /* tp_base */ |
| 427 | 0, /* tp_dict */ |
| 428 | 0, /* tp_descr_get */ |
| 429 | 0, /* tp_descr_set */ |
| 430 | 0, /* tp_dictoffset */ |
| 431 | 0, /* tp_init */ |
| 432 | 0, /* tp_alloc */ |
| 433 | rangeiter_new, /* tp_new */ |
| 434 | }; |
| 435 | |
| 436 | /* Return number of items in range/xrange (lo, hi, step). step > 0 |
| 437 | * required. Return a value < 0 if & only if the true value is too |
| 438 | * large to fit in a signed long. |
| 439 | */ |
| 440 | static long |
| 441 | get_len_of_range(long lo, long hi, long step) |
| 442 | { |
| 443 | /* ------------------------------------------------------------- |
| 444 | If lo >= hi, the range is empty. |
| 445 | Else if n values are in the range, the last one is |
| 446 | lo + (n-1)*step, which must be <= hi-1. Rearranging, |
| 447 | n <= (hi - lo - 1)/step + 1, so taking the floor of the RHS gives |
| 448 | the proper value. Since lo < hi in this case, hi-lo-1 >= 0, so |
| 449 | the RHS is non-negative and so truncation is the same as the |
| 450 | floor. Letting M be the largest positive long, the worst case |
| 451 | for the RHS numerator is hi=M, lo=-M-1, and then |
| 452 | hi-lo-1 = M-(-M-1)-1 = 2*M. Therefore unsigned long has enough |
| 453 | precision to compute the RHS exactly. |
| 454 | ---------------------------------------------------------------*/ |
| 455 | long n = 0; |
| 456 | if (lo < hi) { |
| 457 | unsigned long uhi = (unsigned long)hi; |
| 458 | unsigned long ulo = (unsigned long)lo; |
| 459 | unsigned long diff = uhi - ulo - 1; |
| 460 | n = (long)(diff / (unsigned long)step + 1); |
| 461 | } |
| 462 | return n; |
| 463 | } |
| 464 | |
| 465 | static PyObject * |
| 466 | int_range_iter(long start, long stop, long step) |
| 467 | { |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 468 | rangeiterobject *it = PyObject_New(rangeiterobject, &PyRangeIter_Type); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 469 | if (it == NULL) |
| 470 | return NULL; |
| 471 | it->start = start; |
| 472 | it->step = step; |
| 473 | if (step > 0) |
| 474 | it->len = get_len_of_range(start, stop, step); |
| 475 | else |
| 476 | it->len = get_len_of_range(stop, start, -step); |
| 477 | it->index = 0; |
| 478 | return (PyObject *)it; |
| 479 | } |
| 480 | |
| 481 | static PyObject * |
| 482 | rangeiter_new(PyTypeObject *type, PyObject *args, PyObject *kw) |
| 483 | { |
| 484 | long start, stop, step; |
| 485 | |
| 486 | if (!_PyArg_NoKeywords("rangeiter()", kw)) |
| 487 | return NULL; |
| 488 | |
| 489 | if (!PyArg_ParseTuple(args, "lll;rangeiter() requires 3 int arguments", |
| 490 | &start, &stop, &step)) |
| 491 | return NULL; |
| 492 | |
| 493 | return int_range_iter(start, stop, step); |
| 494 | } |
| 495 | |
| 496 | static PyMethodDef longrangeiter_methods[] = { |
| 497 | {"__length_hint__", (PyCFunction)longrangeiter_len, METH_NOARGS, |
| 498 | length_hint_doc}, |
| 499 | {NULL, NULL} /* sentinel */ |
| 500 | }; |
| 501 | |
| 502 | static void |
| 503 | longrangeiter_dealloc(longrangeiterobject *r) |
| 504 | { |
| 505 | Py_XDECREF(r->index); |
Guido van Rossum | 317e774 | 2007-05-08 15:18:31 +0000 | [diff] [blame] | 506 | Py_XDECREF(r->start); |
| 507 | Py_XDECREF(r->step); |
| 508 | Py_XDECREF(r->len); |
Amaury Forgeot d'Arc | b7f17e4 | 2007-11-15 20:52:21 +0000 | [diff] [blame] | 509 | PyObject_Del(r); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 510 | } |
| 511 | |
| 512 | static PyObject * |
| 513 | longrangeiter_next(longrangeiterobject *r) |
| 514 | { |
| 515 | PyObject *one, *product, *new_index, *result; |
| 516 | if (PyObject_RichCompareBool(r->index, r->len, Py_LT) != 1) |
| 517 | return NULL; |
| 518 | |
| 519 | one = PyLong_FromLong(1); |
| 520 | if (!one) |
| 521 | return NULL; |
| 522 | |
| 523 | product = PyNumber_Multiply(r->index, r->step); |
| 524 | if (!product) { |
| 525 | Py_DECREF(one); |
| 526 | return NULL; |
| 527 | } |
| 528 | |
| 529 | new_index = PyNumber_Add(r->index, one); |
| 530 | Py_DECREF(one); |
| 531 | if (!new_index) { |
| 532 | Py_DECREF(product); |
| 533 | return NULL; |
| 534 | } |
| 535 | |
| 536 | result = PyNumber_Add(r->start, product); |
| 537 | Py_DECREF(product); |
| 538 | if (result) { |
| 539 | Py_DECREF(r->index); |
| 540 | r->index = new_index; |
| 541 | } |
| 542 | |
| 543 | return result; |
| 544 | } |
| 545 | |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 546 | PyTypeObject PyLongRangeIter_Type = { |
Martin v. Löwis | 9f2e346 | 2007-07-21 17:22:18 +0000 | [diff] [blame] | 547 | PyVarObject_HEAD_INIT(&PyType_Type, 0) |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 548 | "longrange_iterator", /* tp_name */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 549 | sizeof(longrangeiterobject), /* tp_basicsize */ |
| 550 | 0, /* tp_itemsize */ |
| 551 | /* methods */ |
| 552 | (destructor)longrangeiter_dealloc, /* tp_dealloc */ |
| 553 | 0, /* tp_print */ |
| 554 | 0, /* tp_getattr */ |
| 555 | 0, /* tp_setattr */ |
Mark Dickinson | e94c679 | 2009-02-02 20:36:42 +0000 | [diff] [blame] | 556 | 0, /* tp_reserved */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 557 | 0, /* tp_repr */ |
| 558 | 0, /* tp_as_number */ |
| 559 | 0, /* tp_as_sequence */ |
| 560 | 0, /* tp_as_mapping */ |
| 561 | 0, /* tp_hash */ |
| 562 | 0, /* tp_call */ |
| 563 | 0, /* tp_str */ |
| 564 | PyObject_GenericGetAttr, /* tp_getattro */ |
| 565 | 0, /* tp_setattro */ |
| 566 | 0, /* tp_as_buffer */ |
| 567 | Py_TPFLAGS_DEFAULT, /* tp_flags */ |
| 568 | 0, /* tp_doc */ |
| 569 | 0, /* tp_traverse */ |
| 570 | 0, /* tp_clear */ |
| 571 | 0, /* tp_richcompare */ |
| 572 | 0, /* tp_weaklistoffset */ |
| 573 | PyObject_SelfIter, /* tp_iter */ |
| 574 | (iternextfunc)longrangeiter_next, /* tp_iternext */ |
| 575 | longrangeiter_methods, /* tp_methods */ |
Raymond Hettinger | 6b27cda | 2005-09-24 21:23:05 +0000 | [diff] [blame] | 576 | 0, |
Raymond Hettinger | 48165d4 | 2002-06-05 20:08:48 +0000 | [diff] [blame] | 577 | }; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 578 | |
| 579 | static PyObject * |
| 580 | range_iter(PyObject *seq) |
| 581 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 582 | rangeobject *r = (rangeobject *)seq; |
| 583 | longrangeiterobject *it; |
Martin v. Löwis | 8445104 | 2007-12-20 22:57:23 +0000 | [diff] [blame] | 584 | long lstart, lstop, lstep; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 585 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 586 | assert(PyRange_Check(seq)); |
Martin v. Löwis | 8445104 | 2007-12-20 22:57:23 +0000 | [diff] [blame] | 587 | |
| 588 | /* If all three fields convert to long, use the int version */ |
| 589 | lstart = PyLong_AsLong(r->start); |
| 590 | if (lstart != -1 || !PyErr_Occurred()) { |
| 591 | lstop = PyLong_AsLong(r->stop); |
| 592 | if (lstop != -1 || !PyErr_Occurred()) { |
| 593 | lstep = PyLong_AsLong(r->step); |
| 594 | if (lstep != -1 || !PyErr_Occurred()) |
| 595 | return int_range_iter(lstart, lstop, lstep); |
| 596 | } |
| 597 | } |
| 598 | /* Some conversion failed, so there is an error set. Clear it, |
| 599 | and try again with a long range. */ |
| 600 | PyErr_Clear(); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 601 | |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 602 | it = PyObject_New(longrangeiterobject, &PyLongRangeIter_Type); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 603 | if (it == NULL) |
| 604 | return NULL; |
Guido van Rossum | 317e774 | 2007-05-08 15:18:31 +0000 | [diff] [blame] | 605 | |
| 606 | /* Do all initialization here, so we can DECREF on failure. */ |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 607 | it->start = r->start; |
Guido van Rossum | 317e774 | 2007-05-08 15:18:31 +0000 | [diff] [blame] | 608 | it->step = r->step; |
| 609 | Py_INCREF(it->start); |
| 610 | Py_INCREF(it->step); |
| 611 | |
| 612 | it->len = it->index = NULL; |
| 613 | |
Mark Dickinson | eb36d31 | 2009-06-24 18:36:46 +0000 | [diff] [blame] | 614 | it->len = range_length_obj(r); |
| 615 | if (!it->len) |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 616 | goto create_failure; |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 617 | it->index = PyLong_FromLong(0); |
| 618 | if (!it->index) |
| 619 | goto create_failure; |
| 620 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 621 | return (PyObject *)it; |
| 622 | |
| 623 | create_failure: |
Guido van Rossum | 317e774 | 2007-05-08 15:18:31 +0000 | [diff] [blame] | 624 | Py_DECREF(it); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 625 | return NULL; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 626 | } |
| 627 | |
| 628 | static PyObject * |
| 629 | range_reverse(PyObject *seq) |
| 630 | { |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 631 | rangeobject *range = (rangeobject*) seq; |
| 632 | longrangeiterobject *it; |
| 633 | PyObject *one, *sum, *diff, *len = NULL, *product; |
Martin v. Löwis | 8445104 | 2007-12-20 22:57:23 +0000 | [diff] [blame] | 634 | long lstart, lstop, lstep; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 635 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 636 | /* XXX(nnorwitz): do the calc for the new start/stop first, |
| 637 | then if they fit, call the proper iter()? |
| 638 | */ |
| 639 | assert(PyRange_Check(seq)); |
Martin v. Löwis | 8445104 | 2007-12-20 22:57:23 +0000 | [diff] [blame] | 640 | |
| 641 | /* If all three fields convert to long, use the int version */ |
| 642 | lstart = PyLong_AsLong(range->start); |
| 643 | if (lstart != -1 || !PyErr_Occurred()) { |
| 644 | lstop = PyLong_AsLong(range->stop); |
| 645 | if (lstop != -1 || !PyErr_Occurred()) { |
| 646 | lstep = PyLong_AsLong(range->step); |
| 647 | if (lstep != -1 || !PyErr_Occurred()) { |
| 648 | /* XXX(nnorwitz): need to check for overflow and simplify. */ |
| 649 | long len = get_len_of_range(lstart, lstop, lstep); |
| 650 | long new_start = lstart + (len - 1) * lstep; |
| 651 | long new_stop = lstart; |
| 652 | if (lstep > 0) |
| 653 | new_stop -= 1; |
| 654 | else |
| 655 | new_stop += 1; |
| 656 | return int_range_iter(new_start, new_stop, -lstep); |
| 657 | } |
| 658 | } |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 659 | } |
Martin v. Löwis | 8445104 | 2007-12-20 22:57:23 +0000 | [diff] [blame] | 660 | PyErr_Clear(); |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 661 | |
Christian Heimes | a22e8bd | 2007-11-29 22:35:39 +0000 | [diff] [blame] | 662 | it = PyObject_New(longrangeiterobject, &PyLongRangeIter_Type); |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 663 | if (it == NULL) |
| 664 | return NULL; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 665 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 666 | /* start + (len - 1) * step */ |
| 667 | len = range_length_obj(range); |
| 668 | if (!len) |
| 669 | goto create_failure; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 670 | |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 671 | one = PyLong_FromLong(1); |
| 672 | if (!one) |
| 673 | goto create_failure; |
| 674 | |
| 675 | diff = PyNumber_Subtract(len, one); |
| 676 | Py_DECREF(one); |
| 677 | if (!diff) |
| 678 | goto create_failure; |
| 679 | |
| 680 | product = PyNumber_Multiply(len, range->step); |
| 681 | if (!product) |
| 682 | goto create_failure; |
| 683 | |
| 684 | sum = PyNumber_Add(range->start, product); |
| 685 | Py_DECREF(product); |
| 686 | it->start = sum; |
| 687 | if (!it->start) |
| 688 | goto create_failure; |
| 689 | it->step = PyNumber_Negative(range->step); |
| 690 | if (!it->step) { |
| 691 | Py_DECREF(it->start); |
| 692 | PyObject_Del(it); |
| 693 | return NULL; |
| 694 | } |
| 695 | |
| 696 | /* Steal reference to len. */ |
| 697 | it->len = len; |
| 698 | |
| 699 | it->index = PyLong_FromLong(0); |
| 700 | if (!it->index) { |
| 701 | Py_DECREF(it); |
| 702 | return NULL; |
| 703 | } |
| 704 | |
| 705 | return (PyObject *)it; |
| 706 | |
| 707 | create_failure: |
| 708 | Py_XDECREF(len); |
| 709 | PyObject_Del(it); |
| 710 | return NULL; |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 711 | } |