I see there is test code here:#1574 (comment)
It would be good to confirm that the test code works on 32bit python and 64bit python.

The return type ofstd::atoll islong long. An alternative solution might bestd::stoll.

Revisiting this PR, I believeatol is the right function to use here. It returns along not along long, which should be the better datatype to use here IG. Because the size of a void pointer is not fixed at 8 bytes likelong long, instead it can vary by implementation. I believe the sizeoflong best matches the size of a void pointer on most platforms and configurations.

As mentioned above, likeatoll, this function can return0 on fail too. The error handling bit can look something like

if (!window_handle) {/* NULL window_handle can segfault when *window_handle executes */window_handle="";}longwindow_id=atol(window_handle);if (!window_id&&*window_handle!='0') {/* window_id is 0 when atol errors. It can also be 0 when the input was 0 */returnRAISE(pgExc_SDLError,"'SDL_WINDOWID' must be an integer");}

As illume pointed above, it would be good to test this change on 32 bit and 64 bit windows, bonus points if this can be put in a unit test ;)

@Rabbid76 are you still interested in finishing this off? It has been a while, so we could find someone else to take this over to get it into 2.1.3.