[GitHub] [qpid-proton] jiridanek commented on a change in pull request #156: PROTON-1936: lower case include and updated ifdefs

[GitBox <gi...@apache.org>]

jiridanek commented on a change in pull request #156:
URL: https://github.com/apache/qpid-proton/pull/156#discussion_r601252880



##########
File path: c/src/platform/platform.h
##########
@@ -81,7 +81,7 @@ int pni_vsnprintf(char *buf, size_t count, const char *fmt, va_list ap);
 
 #endif
 
-#if defined _MSC_VER || defined _OPENVMS
+#if defined _MSC_VER || defined(_WIN32) || defined _OPENVMS

Review comment:
       Note to myself: The switch I mentioned in previous comment already happened, so va_copy on Windows can be relied on.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org