Changes between Initial Version and Version 1 of Ticket #25477, comment 103


Ignore:
Timestamp:
06/19/18 15:06:56 (4 years ago)
Author:
jdemeyer
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #25477, comment 103

    initial v1  
    44}}}
    55
    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.
     62. 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.
    77
    883. Why did you write `lst.__len__()` instead of `len(lst)`?