From 4f3cab39126883a7baaf3a62af677d46083412d8 Mon Sep 17 00:00:00 2001
From: Viacheslav Kroilov <slavakroilov@gmail.com>
Date: Sat, 5 May 2018 19:26:17 +0200
Subject: [PATCH] Fixes for windows

---
 epi_judge_cpp/CMakeLists.txt                     |  2 +-
 epi_judge_cpp/drawing_skyline.cc                 |  8 +++++---
 epi_judge_cpp/rectangle_intersection.cc          | 12 ++++++++----
 epi_judge_cpp/test_framework/binary_tree_utils.h | 16 ++++++++++------
 epi_judge_cpp/test_framework/console_color.h     |  2 ++
 epi_judge_cpp/test_framework/platform.h          |  2 +-
 epi_judge_cpp_solutions/CMakeLists.txt           |  2 +-
 epi_judge_cpp_solutions/drawing_skyline.cc       |  9 ++++++---
 .../rectangle_intersection.cc                    | 14 ++++++++++----
 .../search_unknown_length_array.cc               |  4 ++--
 .../test_framework/binary_tree_utils.h           | 16 ++++++++++------
 .../test_framework/console_color.h               |  2 ++
 .../test_framework/platform.h                    |  2 +-
 13 files changed, 59 insertions(+), 32 deletions(-)

diff --git a/epi_judge_cpp/CMakeLists.txt b/epi_judge_cpp/CMakeLists.txt
index 5ebd80f53..b59d0095d 100644
--- a/epi_judge_cpp/CMakeLists.txt
+++ b/epi_judge_cpp/CMakeLists.txt
@@ -4,7 +4,7 @@ project(epi_judge_cpp)
 set(CMAKE_CXX_STANDARD 14)
 
 if (MSVC)
-    add_definitions(-D_CRT_SECURE_NO_WARNINGS -wd4018 -wd4800 -wd4805)
+    add_definitions(-D_CRT_SECURE_NO_WARNINGS -wd4018 -wd4244 -wd4800 -wd4805)
 else()
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wno-unused-variable -Wno-sign-compare")
     link_libraries(pthread)
diff --git a/epi_judge_cpp/drawing_skyline.cc b/epi_judge_cpp/drawing_skyline.cc
index 54b3609d2..1dfa3d225 100644
--- a/epi_judge_cpp/drawing_skyline.cc
+++ b/epi_judge_cpp/drawing_skyline.cc
@@ -16,12 +16,14 @@ Skyline ComputeSkyline(const vector<Rectangle>& buildings) {
 bool operator==(const Rectangle& a, const Rectangle& b) {
   return a.left == b.left && a.right == b.right && a.height == b.height;
 }
+}  // namespace drawing_skyline
 
 template <>
-struct SerializationTraits<Rectangle>
-    : UserSerTraits<Rectangle, int, int, int> {};
+struct SerializationTraits<drawing_skyline::Rectangle>
+    : UserSerTraits<drawing_skyline::Rectangle, int, int, int> {};
 
-std::ostream& operator<<(std::ostream& out, const Rectangle& r) {
+std::ostream& operator<<(std::ostream& out,
+                         const drawing_skyline::Rectangle& r) {
   return PrintTo(out, std::make_tuple(r.left, r.right, r.height));
 }
 
diff --git a/epi_judge_cpp/rectangle_intersection.cc b/epi_judge_cpp/rectangle_intersection.cc
index ee75363ef..5d642f31e 100644
--- a/epi_judge_cpp/rectangle_intersection.cc
+++ b/epi_judge_cpp/rectangle_intersection.cc
@@ -15,21 +15,25 @@ bool operator==(const Rectangle& r1, const Rectangle& r2) {
          std::tie(r2.x, r2.y, r2.width, r2.height);
 }
 
+}  // namespace rectangle_intersection
+
 template <>
-struct SerializationTraits<Rectangle>
-    : UserSerTraits<Rectangle, int, int, int, int> {
+struct SerializationTraits<rectangle_intersection::Rectangle>
+    : UserSerTraits<rectangle_intersection::Rectangle, int, int, int, int> {
   static std::vector<std::string> GetMetricNames(const std::string& arg_name) {
     return {FmtStr("area({})", arg_name), FmtStr("perimeter({})", arg_name),
             FmtStr("max(w, h)({})", arg_name)};
   }
 
-  static std::vector<int> GetMetrics(const Rectangle& x) {
+  static std::vector<int> GetMetrics(
+      const rectangle_intersection::Rectangle& x) {
     return {x.height * x.width, 2 * (x.height + x.width),
             std::max(x.height, x.width)};
   }
 };
 
-std::ostream& operator<<(std::ostream& out, const Rectangle& r) {
+std::ostream& operator<<(std::ostream& out,
+                         const rectangle_intersection::Rectangle& r) {
   return PrintTo(out, std::make_tuple(r.x, r.y, r.width, r.height));
 }
 
diff --git a/epi_judge_cpp/test_framework/binary_tree_utils.h b/epi_judge_cpp/test_framework/binary_tree_utils.h
index c0c5afa8b..43d661c3f 100644
--- a/epi_judge_cpp/test_framework/binary_tree_utils.h
+++ b/epi_judge_cpp/test_framework/binary_tree_utils.h
@@ -29,25 +29,29 @@ void TreeGenerateHelper(const Node& tree, std::vector<T>* result, int order) {
   }
 }
 
+//TODO Try type extractor pattern
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GeneratePreorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GeneratePreorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, -1);
   return result;
 }
 
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GenerateInorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GenerateInorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, 0);
   return result;
 }
 
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GeneratePostorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GeneratePostorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, 1);
   return result;
diff --git a/epi_judge_cpp/test_framework/console_color.h b/epi_judge_cpp/test_framework/console_color.h
index 489c66f32..cbd0ae6e2 100644
--- a/epi_judge_cpp/test_framework/console_color.h
+++ b/epi_judge_cpp/test_framework/console_color.h
@@ -5,7 +5,9 @@
 #include <iostream>
 
 #ifdef PLATFORM_WIN
+#ifndef NOMINMAX
 #define NOMINMAX
+#endif
 #include <Windows.h>
 #endif
 
diff --git a/epi_judge_cpp/test_framework/platform.h b/epi_judge_cpp/test_framework/platform.h
index 9919f7358..58a745a12 100644
--- a/epi_judge_cpp/test_framework/platform.h
+++ b/epi_judge_cpp/test_framework/platform.h
@@ -6,7 +6,7 @@
  * function.
  */
 
-#if _WINDOWS || __MINGW__ || __CYGWIN__
+#if _WINDOWS || __MINGW32__ || __CYGWIN__
 #define PLATFORM_WIN
 #endif
 
diff --git a/epi_judge_cpp_solutions/CMakeLists.txt b/epi_judge_cpp_solutions/CMakeLists.txt
index 18e22f731..8e041caff 100644
--- a/epi_judge_cpp_solutions/CMakeLists.txt
+++ b/epi_judge_cpp_solutions/CMakeLists.txt
@@ -4,7 +4,7 @@ project(epi_judge_cpp_solutions)
 set(CMAKE_CXX_STANDARD 17)
 
 if (MSVC)
-    add_definitions(-D_CRT_SECURE_NO_WARNINGS -wd4018 -wd4800 -wd4805)
+    add_definitions(-D_CRT_SECURE_NO_WARNINGS -wd4018 -wd4244 -wd4800 -wd4805)
 else()
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wno-unused-variable -Wno-sign-compare")
     link_libraries(pthread)
diff --git a/epi_judge_cpp_solutions/drawing_skyline.cc b/epi_judge_cpp_solutions/drawing_skyline.cc
index 71432e590..2b6506f16 100644
--- a/epi_judge_cpp_solutions/drawing_skyline.cc
+++ b/epi_judge_cpp_solutions/drawing_skyline.cc
@@ -10,6 +10,7 @@ using std::min;
 using std::numeric_limits;
 using std::vector;
 
+namespace drawing_skyline {
 struct Rectangle;
 typedef vector<Rectangle> Skyline;
 
@@ -49,12 +50,14 @@ Skyline ComputeSkyline(const vector<Rectangle>& buildings) {
 bool operator==(const Rectangle& a, const Rectangle& b) {
   return a.left == b.left && a.right == b.right && a.height == b.height;
 }
+}  // namespace drawing_skyline
 
 template <>
-struct SerializationTraits<Rectangle>
-    : UserSerTraits<Rectangle, int, int, int> {};
+struct SerializationTraits<drawing_skyline::Rectangle>
+    : UserSerTraits<drawing_skyline::Rectangle, int, int, int> {};
 
-std::ostream& operator<<(std::ostream& out, const Rectangle& r) {
+std::ostream& operator<<(std::ostream& out,
+                         const drawing_skyline::Rectangle& r) {
   return PrintTo(out, std::make_tuple(r.left, r.right, r.height));
 }
 
diff --git a/epi_judge_cpp_solutions/rectangle_intersection.cc b/epi_judge_cpp_solutions/rectangle_intersection.cc
index 88bb389a9..861d7765d 100644
--- a/epi_judge_cpp_solutions/rectangle_intersection.cc
+++ b/epi_judge_cpp_solutions/rectangle_intersection.cc
@@ -8,7 +8,9 @@
 using std::max;
 using std::min;
 
+namespace rectangle_intersection {
 struct Rectangle;
+
 bool IsIntersect(const Rectangle&, const Rectangle&);
 
 struct Rectangle {
@@ -34,21 +36,25 @@ bool operator==(const Rectangle& r1, const Rectangle& r2) {
          std::tie(r2.x, r2.y, r2.width, r2.height);
 }
 
+}  // namespace rectangle_intersection
+
 template <>
-struct SerializationTraits<Rectangle>
-    : UserSerTraits<Rectangle, int, int, int, int> {
+struct SerializationTraits<rectangle_intersection::Rectangle>
+    : UserSerTraits<rectangle_intersection::Rectangle, int, int, int, int> {
   static std::vector<std::string> GetMetricNames(const std::string& arg_name) {
     return {FmtStr("area({})", arg_name), FmtStr("perimeter({})", arg_name),
             FmtStr("max(w, h)({})", arg_name)};
   }
 
-  static std::vector<int> GetMetrics(const Rectangle& x) {
+  static std::vector<int> GetMetrics(
+      const rectangle_intersection::Rectangle& x) {
     return {x.height * x.width, 2 * (x.height + x.width),
             std::max(x.height, x.width)};
   }
 };
 
-std::ostream& operator<<(std::ostream& out, const Rectangle& r) {
+std::ostream& operator<<(std::ostream& out,
+                         const rectangle_intersection::Rectangle& r) {
   return PrintTo(out, std::make_tuple(r.x, r.y, r.width, r.height));
 }
 
diff --git a/epi_judge_cpp_solutions/search_unknown_length_array.cc b/epi_judge_cpp_solutions/search_unknown_length_array.cc
index b4b2663d8..1d22b8c18 100644
--- a/epi_judge_cpp_solutions/search_unknown_length_array.cc
+++ b/epi_judge_cpp_solutions/search_unknown_length_array.cc
@@ -16,7 +16,7 @@ int BinarySearchUnknownLength(const vector<int>& A, int k) {
       } else if (A.at(idx) > k) {
         break;
       }
-    } catch (const exception& e) {
+    } catch (const exception&) {
       break;
     }
     ++p;
@@ -34,7 +34,7 @@ int BinarySearchUnknownLength(const vector<int>& A, int k) {
       } else {  // A.at(mid) < k
         left = mid + 1;
       }
-    } catch (const exception& e) {
+    } catch (const exception&) {
       right = mid - 1;  // Search the left part if out-of-bound.
     }
   }
diff --git a/epi_judge_cpp_solutions/test_framework/binary_tree_utils.h b/epi_judge_cpp_solutions/test_framework/binary_tree_utils.h
index c0c5afa8b..43d661c3f 100644
--- a/epi_judge_cpp_solutions/test_framework/binary_tree_utils.h
+++ b/epi_judge_cpp_solutions/test_framework/binary_tree_utils.h
@@ -29,25 +29,29 @@ void TreeGenerateHelper(const Node& tree, std::vector<T>* result, int order) {
   }
 }
 
+//TODO Try type extractor pattern
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GeneratePreorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GeneratePreorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, -1);
   return result;
 }
 
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GenerateInorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GenerateInorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, 0);
   return result;
 }
 
 template <template <typename...> class SmartPtr,
-          template <typename...> class Node, typename T>
-std::vector<T> GeneratePostorder(const SmartPtr<Node<T>>& tree) {
+          template <typename...> class Node, typename T,
+          typename... MsvcWorkaround>
+std::vector<T> GeneratePostorder(const SmartPtr<Node<T>, MsvcWorkaround...>& tree) {
   std::vector<T> result;
   TreeGenerateHelper(tree, &result, 1);
   return result;
diff --git a/epi_judge_cpp_solutions/test_framework/console_color.h b/epi_judge_cpp_solutions/test_framework/console_color.h
index 489c66f32..cbd0ae6e2 100644
--- a/epi_judge_cpp_solutions/test_framework/console_color.h
+++ b/epi_judge_cpp_solutions/test_framework/console_color.h
@@ -5,7 +5,9 @@
 #include <iostream>
 
 #ifdef PLATFORM_WIN
+#ifndef NOMINMAX
 #define NOMINMAX
+#endif
 #include <Windows.h>
 #endif
 
diff --git a/epi_judge_cpp_solutions/test_framework/platform.h b/epi_judge_cpp_solutions/test_framework/platform.h
index 9919f7358..58a745a12 100644
--- a/epi_judge_cpp_solutions/test_framework/platform.h
+++ b/epi_judge_cpp_solutions/test_framework/platform.h
@@ -6,7 +6,7 @@
  * function.
  */
 
-#if _WINDOWS || __MINGW__ || __CYGWIN__
+#if _WINDOWS || __MINGW32__ || __CYGWIN__
 #define PLATFORM_WIN
 #endif