<URL: https://rt.cpan.org/Ticket/Display.html?id=77525 >
I have read most of your patch. It's somewhat difficult to digest all the changes together. I think it would have been easier to get a series of commits, maybe in a forked git repository (hint :) I'm impressed by the amount of work you did. I'd like someone else to have a look at this patch, but my feedback is: * Not sure the Win32::API/Win32::API::More namespace split is the best solution. I'd suggest reaching out to Win32::API users to estimate how much work it would be to change "Ss" for short ints. * Please avoid the WIN32_API_SORRY_I_WAS_AN_IDIOT variable, and rename it to something that explains the purpose of it.