Remove a couple of MSVC portability defines

Summary: The first is both wrong (due to the difference in semantics) and unneeded, as MSVC 2015 supports `snprintf` natively and with the correct semantics. The second is simply never used.

Reviewed By: yfeldblum

Differential Revision: D3387551

fbshipit-source-id: ea666a60ce859a59ad3d3ff961b4d7d642fd86eb

diff --git a/folly/Portability.h b/folly/Portability.h
index 55c5dd5ca8b9a11f818acd0cfda8dd82338b5843..61c698469f3d305575d17ea6a56b40cfc689fb51 100644 (file)
--- a/folly/Portability.h
+++ b/folly/Portability.h
@@ -256,13 +256,6 @@ namespace std { typedef ::max_align_t max_align_t; }
 #ifdef _MSC_VER
 #include <folly/portability/SysTypes.h>
 
-// sprintf semantics are not exactly identical
-// but current usage is not a problem
-# define snprintf _snprintf
-
-// semantics here are identical
-# define strerror_r(errno,buf,len) strerror_s(buf,len,errno)
-
 // compiler specific to compiler specific
 // nolint
 # define __PRETTY_FUNCTION__ __FUNCSIG__