gh-111178: fix UBSan failures in Modules/overlapped.c - #129786
Conversation
|
Yeah, the whole file To get rid of the warnings, I think, all those I assume, there you prefixed with Maybe instead change the return type of |
|
No, the problem here is that |
|
Sure! Having another thought, then maybe not change Is this possible? |
No that's the problem of the PR. The signature of the clear function must be |
|
Yeah, I know. But AFAICT In contrast to e.g. |
|
OH! There wasn't an issue with the clear! I... missed the comment... oook. I'll improve the comment as well and will shorten the PR. My bad! Thanks for noticing this! |

This PR fixes the UBSan failures and addresses some minor cosmetic changes. PEP-7 changes were not applied since they could scramble the diff but other semantic changes affecting the signature of touched functions may have been done.