i think the PR is fine-- just one minor comment wrt a

GetLastError

call. I assume what will happen now is

shutdownNow(EXIT_CATASTROPHIC)

if it isn't able to timely shutdown. From a user's perspective, if this specific scenario plays out, they will observe a notification that the service shut down unexpectedly. This case might be acceptable. It isn't OPT but it's a better feasible solution than what is now. 🙂