Changes between Initial Version and Version 1 of Ticket #25477, comment 103
- Timestamp:
- 06/19/18 15:06:56 (4 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #25477, comment 103
initial v1 4 4 }}} 5 5 6 2. Are you certain that `cdef int vn = lst.__len__()` won't overflow? As far as I know, libGAP supports lists with over `2^32^`elements. Better use `Py_ssize_t`, which is anyway the standard CPython type for lengths.6 2. Are you certain that `cdef int vn = lst.__len__()` won't overflow? As far as I know, both Python and libGAP support lists with over 2^32^ elements. Better use `Py_ssize_t`, which is anyway the standard CPython type for lengths. 7 7 8 8 3. Why did you write `lst.__len__()` instead of `len(lst)`?