Modernize code base

.clang-format

@@ -0,0 +1,23 @@
+# Use the Google style in this project.
+BasedOnStyle: Google
+
+# Some folks prefer to write "int& foo" while others prefer "int &foo".  The
+# Google Style Guide only asks for consistency within a project, we chose
+# "int& foo" for this project:
+DerivePointerAlignment: false
+PointerAlignment: Left
+
+# Rules for includes
+# IncludeBlocks: IBS_Regroup # Requires clang-format 6.0+
+# Ordering of includes, higher priority lower in list
+IncludeCategories:
+- Regex: '^\"unit/'
+  Priority: 5999
+- Regex: '^\"'
+  Priority: 4000
+- Regex: '^<.*/.*'
+  Priority: 2000
+- Regex: '^<[^\/.]*>'
+  Priority: 1000
+- Regex: '^<.*.h>'
+  Priority: 500

.clang-tidy

@@ -0,0 +1,42 @@
+---
+# Configure clang-tidy for this project.
+
+# Disabled:
+#  -clang-analyzer-cplusplus due to errors with the stl
+#  -google-readability-namespace-comments due to c++17 one line multinamespace
+#  -bugprone-suspicius-include does not support clang-tidy build styles
+#  -bugprone-easily-swappable-parameters often disallows simple functions
+#  -readability-function-cognitive-complexity disallows large functions
+#  -readability-identifier-length allows short variable names
+# To be corrected:
+Checks: '-*, bugprone-*, clang-*, google-readability-*, misc-*, modernize-*, readability-*, performance-*, -clang-analyzer-cplusplus*, -google-readability-namespace-comments, -bugprone-easily-swappable-parameters, -bugprone-suspicious-include, -readability-function-cognitive-complexity, -readability-identifier-length, -modernize-use-trailing-return-type'
+# Most warnings should be errors
+WarningsAsErrors: 'bugprone-*, clang-*, google-*, misc-*, modernize-*, readability-*, performance-*'
+# Overall format style, Google Style Guide w/ extra restrictions
+FormatStyle:      file
+CheckOptions:
+  - { key: readability-identifier-naming.ClassCase,                value: CamelCase  }
+  - { key: readability-identifier-naming.StructCase,               value: CamelCase  }
+  - { key: readability-identifier-naming.TemplateParameterCase,    value: CamelCase  }
+  - { key: readability-identifier-naming.FunctionCase,             value: CamelCase  }
+  - { key: readability-identifier-naming.NamespaceCase,            value: lower_case }
+  - { key: readability-identifier-naming.VariableCase,             value: lower_case }
+  - { key: readability-identifier-naming.LocalConstantCase,        value: lower_case }
+  - { key: readability-identifier-naming.LocalConstantCase,        value: lower_case }
+  - { key: readability-identifier-naming.GlobalVariableCase,       value: lower_case }
+  - { key: readability-identifier-naming.GlobalVariableSuffix,     value: _          }
+  - { key: readability-identifier-naming.GlobalConstantCase,       value: CamelCase  }
+  - { key: readability-identifier-naming.GlobalConstantPrefix,     value: k          }
+  - { key: readability-identifier-naming.PrivateMemberSuffix,      value: _          }
+  - { key: readability-identifier-naming.PrivateMemberCase,        value: lower_case }
+  - { key: readability-identifier-naming.ProtectedMemberSuffix,    value: _          }
+  - { key: readability-identifier-naming.ProtectedMemberCase,      value: lower_case }
+  - { key: readability-identifier-naming.ConstexprVariableCase,    value: CamelCase  }
+  - { key: readability-identifier-naming.ConstexprVariablePrefix,  value: k          }
+  - { key: readability-identifier-naming.EnumConstantCase,         value: CamelCase  }
+  - { key: readability-identifier-naming.EnumConstantPrefix,       value: k          }
+  - { key: readability-identifier-naming.MemberConstantCase,       value: CamelCase  }
+  - { key: readability-identifier-naming.MemberConstantPrefix,     value: k          }
+  - { key: readability-identifier-naming.StaticConstantCase,       value: CamelCase  }
+  - { key: readability-identifier-naming.StaticConstantPrefix,     value: k          }
+  - { key: readability-function-cognitive-complexity.Threshold,    value: 35         }

.gitignore

@@ -14,4 +14,4 @@ CMakeFiles
 cmake_install.cmake
 CPackConfig.cmake
 CPackSourceConfig.cmake
-
+.cache

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.2)
+cmake_minimum_required(VERSION 3.22)
 project(OpenDIS VERSION 1.0.1)
 
 # Build options
@@ -7,10 +7,12 @@ option(BUILD_EXAMPLES "build example sender/receiver applications" OFF)
 option(BUILD_TESTS "build unit tests" OFF)
 
 # Enforce a C++11 compliant compiler
-set(CMAKE_CXX_STANDARD 11)
+set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 # include GNUInstallDirs Module to get more generic directory handling
 include(GNUInstallDirs)
 
@@ -19,9 +21,14 @@ add_library(OpenDIS6 "")
 
 # Add src/ to the include directories to OpenDIS6
 target_include_directories(OpenDIS6
-  PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src>
-         $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/src>
-         $<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>)
+  PUBLIC $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/src>
+    $<INSTALL_INTERFACE:${INSTALL_INCLUDE_DIR}>
+)
+target_include_directories(OpenDIS6 SYSTEM
+  PUBLIC $<BUILD_INTERFACE:${PROJECT_BINARY_DIR/src}>
+    $<INSTALL_INTERFACE:${INSTALL_INCLUDE_DIR}>
+  PRIVATE $<BUILD_INTERFACE:${CMRC_INCLUDE_DIR}>
+)
 
 # Define OpenDIS7 Library
 add_library(OpenDIS7 "")