[rheard]

Fixes mypyc/mypyc#1235.

native_slot() filled tp_iternext with the native __next__ method itself, which only works if the method returns PyObject *. With -> int, CPython got the tagged integer, or CPY_INT_TAG after StopIteration, as an object pointer, so next(), for loops, list() and obj.__next__() from Python all crashed. float, i64 and tuples crashed too. With bool, False came back as NULL, so the iteration stopped early without an error. MSVC warns about the mismatch (C4047).

native_slot() now uses generate_dunder_wrapper() when the method returns an unboxed value. That's the wrapper the unary number slots such as __int__ and __index__ already use. It calls the native method, returns NULL if the method raised (checking PyErr_Occurred() for types such as float, whose error value is also a valid value), and boxes the result. A StopIteration raised by __next__ still ends the iteration, because the wrapper returns NULL with it set. Methods that return an object, including the __next__ of every native generator, still fill the slot directly, so they don't get an extra call. The other slots that use native_slot() (__iter__, __str__, __repr__, __await__, __aiter__ and __anext__) get the same check, although they normally return objects.

[JukkaL]

On master, it failed to compile for me instead of segfaulting (due to -Werror), but otherwise looks good. I'll update the description and title based on this.