Another fix for newer Windows SDK

Summary: This issue was only showing up in a single test, but include orders meant that stdio.h could end up being included in a way that caused it to not properly define the normal names it's supposed to, as they were getting disabled due to the `_CRT_INTERNAL_NON_STDC_NAMES` changes being made.

Reviewed By: yfeldblum

Differential Revision: D19866950

fbshipit-source-id: 9d6391f5c0e5a2ef44f0f8ba6dad064cfecf0a82

folly/portability/Windows.h

@@ -26,6 +26,12 @@
 // These have to be this way because we define our own versions
 // of close(), because the normal Windows versions don't handle
 // sockets at all.
+
+// There are some ordering issues internally in the SDK; we need to ensure
+// stdio.h is included prior to including direct.h and io.h with internal names
+// disabled to ensure all of the normal names get declared properly.
+#include <stdio.h>
+
 #ifndef __STDC__
 /* nolint */
 #define __STDC__ 1