Support building on MSVC

CMakeLists.txt

@@ -21,19 +21,17 @@ include(cmake/SettiUtils.cmake)
 include(cmake/sanitize-helpers.cmake)
 include(CheckCXXCompilerFlag)
 
-CHECK_CXX_COMPILER_FLAG("-std=c++14" COMPILER_SUPPORTS_CXX14)
-if(COMPILER_SUPPORTS_CXX14)
-  set(CMAKE_CXX_STANDARD 14)
-  set(CMAKE_CXX_STANDARD_REQUIRED ON)
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14")
+add_library(project_options INTERFACE)
+target_compile_features(project_options INTERFACE cxx_std_14)
+
+include(cmake/CompilerWarnings.cmake)
+
+if(MSVC)
 else()
-  message(STATUS "The compiler ${CMAKE_CXX_COMPILER} has no C++14 support. Please use a different C++ compiler.")
+  set(CMAKE_CXX_FLAGS_DEBUG "-g")
+  set(CMAKE_CXX_FLAGS_RELEASE "-O3")
 endif()
 
-set(CMAKE_CXX_FLAGS "-Wall -Wextra")
-set(CMAKE_CXX_FLAGS_DEBUG "-g")
-set(CMAKE_CXX_FLAGS_RELEASE "-O3")
-
 SET(CMAKE_SKIP_BUILD_RPATH  FALSE)
 
 SET(CMAKE_BUILD_WITH_INSTALL_RPATH FALSE)

cmake/CompilerWarnings.cmake

@@ -0,0 +1,24 @@
+# Set compiler warnings
+add_library(project_warnings INTERFACE)
+
+set(MSVC_WARNINGS
+    /W4)
+
+set(CLANG_WARNINGS
+    -Wall
+    -Wextra)
+
+set(GCC_WARNINGS
+    ${CLANG_WARNINGS})
+
+if(MSVC)
+  set(PROJECT_WARNINGS ${MSVC_WARNINGS})
+elseif(CMAKE_CXX_COMPILER_ID MATCHES ".*Clang")
+  set(PROJECT_WARNINGS ${CLANG_WARNINGS})
+elseif(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
+  set(PROJECT_WARNINGS ${GCC_WARNINGS})
+else()
+  message(AUTHOR_WARNING "No compiler warnings set for '${CMAKE_CXX_COMPILER_ID}' compiler.")
+endif()
+
+target_compile_options(project_warnings INTERFACE ${PROJECT_WARNINGS})

src/ast/ast.h

@@ -168,16 +168,16 @@ class Expression;
 
 // Position of ast node on source code
 struct Position {
-  uint line;
-  uint col;
+  unsigned int line;
+  unsigned int col;
 };
 
 const char* AstNodeStr(size_t i);
 
 class AstNode {
  public:
 #define DECLARE_TYPE_ENUM(type) k##type,
-  enum class NodeType : uint8_t { AST_NODE_LIST(DECLARE_TYPE_ENUM) };
+  enum class NodeType : unsigned int8_t { AST_NODE_LIST(DECLARE_TYPE_ENUM) };
 #undef DECLARE_TYPE_ENUM
 
   virtual ~AstNode() {}

src/msg.h

@@ -33,7 +33,7 @@ class Message {
 
   Message() = delete;
 
-  Message(Severity severity, const boost::format& msg, uint line, uint pos)
+  Message(Severity severity, const boost::format& msg, unsigned int line, unsigned int pos)
       : severity_(severity)
       , msg_(msg)
       , line_(line)
@@ -62,11 +62,11 @@ class Message {
     return msg_.str();
   }
 
-  uint line() {
+  unsigned int line() {
     return line_;
   }
 
-  uint pos() {
+  unsigned int pos() {
     return pos_;
   }
 
@@ -92,8 +92,8 @@ class Message {
  private:
   Severity severity_;
   boost::format msg_;
-  uint line_;
-  uint pos_;
+  unsigned int line_;
+  unsigned int pos_;
   std::string str_line_error_;
   std::string file_;
 };

src/objects/abstract-obj.h

@@ -34,7 +34,7 @@ class Object {
   using Args = std::vector<std::shared_ptr<Object>>;
   using KWArgs = std::unordered_map<std::string, std::shared_ptr<Object>>;
 
-  enum class ObjectType: uint8_t {
+  enum class ObjectType: unsigned int8_t {
     ROOT,
     NIL,
     INT,

src/parser/lexer.cc

@@ -56,7 +56,7 @@ std::string Lexer::ScanUnicodeEscapeCode() {
 
   Back();
 
-  uint32_t hex_number = static_cast<uint32_t>(std::stoi(number,nullptr,16));
+  unsigned int32_t hex_number = static_cast<unsigned int32_t>(std::stoi(number,nullptr,16));
 
   // initialize wstring with 1 wchar_t
   std::wstring str_hex(1, static_cast<wchar_t>(hex_number));

src/parser/lexer.h

@@ -42,7 +42,7 @@ class Lexer {
 
   TokenStream Scanner();
 
-  inline uint NumErrors() noexcept {
+  inline unsigned int NumErrors() noexcept {
     return nerror_;
   }
 
@@ -174,13 +174,13 @@ class Lexer {
   void ErrorMsg(const boost::format& fmt_msg);
 
   std::string str_;
-  uint strlen_;
+  unsigned int strlen_;
   char c_;
-  uint buffer_cursor_;
-  uint line_;
-  uint line_pos_;
-  uint start_pos_;
-  uint nerror_;
+  unsigned int buffer_cursor_;
+  unsigned int line_;
+  unsigned int line_pos_;
+  unsigned int start_pos_;
+  unsigned int nerror_;
   Messages msgs_;
 
 };

src/parser/parser.h

@@ -50,7 +50,7 @@ class Parser {
     return ParserExpCmd();
   }
 
-  inline uint nerrors() const {
+  inline unsigned int nerrors() const {
     return nerror_;
   }
 
@@ -350,7 +350,7 @@ class Parser {
 
   TokenStream ts_;
   AstNodeFactory factory_;
-  uint nerror_;
+  unsigned int nerror_;
   Token& token_;
   Token token_error_;
   Messages msgs_;

src/parser/token.h

@@ -73,21 +73,21 @@ class Token {
  public:
   using Value = boost::variant<int, std::string, float, bool>;
 
-  Token(TokenKind k, Value value, bool blank_after, uint line, uint col)
+  Token(TokenKind k, Value value, bool blank_after, unsigned int line, unsigned int col)
       : kind_(k)
       , value_(value)
       , blank_after_(blank_after)
       , line_(line)
       , col_(col) {}
 
-  Token(TokenKind k, const char* value, bool blank_after, uint line, uint col)
+  Token(TokenKind k, const char* value, bool blank_after, unsigned int line, unsigned int col)
       : kind_(k)
       , value_(std::string(value))
       , blank_after_(blank_after)
       , line_(line)
       , col_(col) {}
 
-  Token(TokenKind k, bool blank_after, uint line, uint col)
+  Token(TokenKind k, bool blank_after, unsigned int line, unsigned int col)
       : kind_(k)
       , value_(std::string(token_value_str[static_cast<int>(k)]))
       , blank_after_(blank_after)
@@ -143,9 +143,9 @@ class Token {
 
   bool BlankAfter() const noexcept { return blank_after_; }
 
-  uint Line() const noexcept { return line_; }
+  unsigned int Line() const noexcept { return line_; }
 
-  uint Col() const noexcept { return col_; }
+  unsigned int Col() const noexcept { return col_; }
 
   bool Is(TokenKind k) const noexcept { return kind_ == k; }
 
@@ -264,8 +264,8 @@ class Token {
   TokenKind kind_;
   Value value_;
   bool blank_after_;
-  uint line_;
-  uint col_;
+  unsigned int line_;
+  unsigned int col_;
 };
 
 inline std::ostream& operator<<(std::ostream& stream, Token& token) {

src/run_time_error.h

@@ -37,7 +37,7 @@ namespace internal {
  */
 class RunTimeError : public std::exception {
  public:
-  enum class ErrorCode: uint8_t{
+  enum class ErrorCode: unsigned int8_t{
     NULL_ACCESS,
     SYMBOL_NOT_FOUND,
     BAD_ALLOC,