Fix Windows warnings

Summary: We want to ignore the CRT SECURE warnings for our test usage.  Additionally, we had one more use of make_shared with a type that had strict alignment requirements.  This is still a bug for C++14, so fix this.

Reviewed By: RomanFedotovFB

Differential Revision: D54215051

fbshipit-source-id: ab2e4e42441a57f012e68db7b17ebc1b021046a1

Author: graphicsMan

dispenso/platform.h

@@ -13,6 +13,7 @@
 #include <algorithm>
 #include <atomic>
 #include <cstdlib>
+#include <memory>
 #include <thread>
 #include <type_traits>
 
@@ -167,6 +168,13 @@ struct AlignedFreeDeleter<void> {
   }
 };
 
+template <typename T, class... Args>
+std::shared_ptr<T> make_shared(Args&&... args) {
+  void* tv = alignedMalloc(sizeof(T), alignof(T));
+  T* t = new (tv) T(std::forward<Args>(args)...);
+  return std::shared_ptr<T>(t, AlignedFreeDeleter<T>());
+}
+
 inline constexpr uintptr_t alignToCacheLine(uintptr_t val) {
   constexpr uintptr_t kMask = kCacheLineSize - 1;
   val += kMask;

dispenso/timed_task.h

@@ -104,7 +104,7 @@ class TimedTask {
       double period = 0.0,
       size_t timesToRun = 1,
       TimedTaskType type = TimedTaskType::kNormal)
-      : impl_(std::make_shared<detail::TimedTaskImpl>(
+      : impl_(detail::make_shared<detail::TimedTaskImpl>(
             timesToRun,
             nextRunAbs,
             period,

tests/timed_task_test.cpp

@@ -5,6 +5,10 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+#if defined(_WIN32) && !defined(_CRT_SECURE_NO_WARNINGS)
+#define _CRT_SECURE_NO_WARNINGS // We don't want warnings regarding the getenv use in this test.
+#endif
+
 #include <dispenso/completion_event.h>
 #include <dispenso/schedulable.h>
 #include <dispenso/task_set.h>