win: suppress GetVersionEx deprecation warning

piscisaureus · piscisaureus · commit 09fe6fe153db · 2015-01-08T14:17:23.000+01:00
There is no other way to retrieve the Windows version. The stated reason this API is deprecated is that applications are not supposed to check whether the Windows it's running on is recent enough. But that's not what we use it for. PR-URL: nodejs#261 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
diff --git a/src/node_os.cc b/src/node_os.cc
@@ -111,6 +111,9 @@ static void GetOSRelease(const FunctionCallbackInfo<Value>& args) {
   OSVERSIONINFOW info;
 
   info.dwOSVersionInfoSize = sizeof(info);
+
+  // Don't complain that GetVersionEx is deprecated; there is no alternative.
+  #pragma warning(suppress : 4996)
   if (GetVersionExW(&info) == 0)
     return;
 
