From: Christopher Dykes <cdykes@fb.com>
Date: Sat, 4 Jun 2016 00:42:35 +0000 (-0700)
Subject: Remove a couple of MSVC portability defines
X-Git-Tag: 2016.07.26~166
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=65b5c3d26e0b15814cfa0358c46f5335586bc48a;p=folly.git

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 55c5dd5c..61c69846 100644
--- 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__