Rewritten parts of the code for better performance. Removed usage of …

….at since 20% of code time was spend inside that function and removed nice abstraction of graph and moved to a simple adjacency list representation
rowanG077 · Nov 14, 2018 · ebee28b · ebee28b
1 parent a56672a
commit ebee28b
Show file tree

Hide file tree

Showing 26 changed files with 148 additions and 1,099 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -1,4 +1,4 @@
-Checks: '*,-google-runtime-references,-fuchsia-default-arguments,-google-readability-namespace-comments,-hicpp-deprecated-headers,-llvm-header-guard,-llvm-namespace-comment,-modernize-deprecated-headers,-fuchsia-overloaded-operator'
+Checks: '*,-google-runtime-references,-fuchsia-default-arguments,-clang-diagnostic-sign-compare,-google-readability-namespace-comments,-cppcoreguidelines-pro-bounds-constant-array-index,-hicpp-deprecated-headers,-llvm-header-guard,-llvm-namespace-comment,-modernize-deprecated-headers,-fuchsia-overloaded-operator'
 WarningsAsErrors: ''
 HeaderFilterRegex: '.*'
 AnalyzeTemporaryDtors: false

diff --git a/cmake/code.cmake b/cmake/code.cmake
@@ -7,11 +7,14 @@ set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_EXTENSIONS ON)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
+# disable sign compare since we use only signed integers types
+add_compile_options(-Wno-sign-compare)
+
 # set warning flags for debug
 if(CMAKE_BUILD_TYPE STREQUAL "Debug")
 	if(CMAKE_C_COMPILER_ID STREQUAL "Clang" OR
 		CMAKE_C_COMPILER_ID STREQUAL "GNU")
-		add_compile_options(-Wall -Wextra -Wconversion)
+		add_compile_options(-Wall -Wextra)
 		if(${WERROR})
 			add_compile_options(-Werror)
 		endif()

diff --git a/include/BoxNesting/Algorithm.hpp b/include/BoxNesting/Algorithm.hpp
diff --git a/include/BoxNesting/Box.hpp b/include/BoxNesting/Box.hpp
diff --git a/include/BoxNesting/Parser.hpp b/include/BoxNesting/Parser.hpp
diff --git a/include/BoxNesting/ParserError.hpp b/include/BoxNesting/ParserError.hpp