Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems wrong to have it present for types that do not implement numbers protocol.

From the code you linked it looks like our types would pass PyNumber_Check. - no it does not, as nb_index, nb_float or nb_int are not set in this function.

Also, what is the relationship between sq_length and tp_as_sequence and why does one get set to another? And the same for the other two fields...