I had to constantly skip back and forth to figure out where one word starts and another ends. Very painful to read, I've found new appreciation for whitespace.
Surprisingly legible, but feels like I can only read it with momentum, flitting past it and letting my subconscious tell me where the word breaks are. The moment I get confused and look more closely, it becomes almost impossible to read.
Note that while Visual C++'s msvcrt doesn't implement this POSIX function officially, there's a nonstandard _ofcyfpos_s() and it will in fact warn you that any use of the official ofcyfpos() is unsafe. The semantics are slightly different (it'll return 1 on success instead of the length of the reply) so you can't just #define the problem away.
Don't forget to set the cbSize of the GETWITTYREPLYEXINFO structure before passing it to GetWittyReplyEx() or you'll get funny things happening to your stack!
That's handled by virtue of GetWittyReplyEx being #defined to GetWittyReplyExA and GetWittyReplyExW right? Just be aware that nMaxReplyMessage needs to be specified in bytes (excluding the null terminator!) but the returned length is in characters.
i've definitely found the ascii version of a syscall being called because that's what the linux project uses so why wouldn't the junior dev that was assigned the port do it too?
there's no #define that will save you from that. I may or may not have been that junior dev so no shade.