run clang-tidy with C++17

Author: firewave

.clang-tidy

@@ -43,8 +43,10 @@ Checks: >
         -modernize-deprecated-ios-base-aliases,
         -misc-include-cleaner,
         -misc-unused-using-decls,
+        -modernize-avoid-bind,
         -modernize-loop-convert,
         -modernize-macro-to-enum,
+        -modernize-make-unique,
         -modernize-raw-string-literal,
         -modernize-replace-auto-ptr,
         -modernize-return-braced-init-list,

clang-tidy.md

@@ -133,6 +133,7 @@ Does not improve the readability.
 `misc-unconventional-assign-operator`<br/>
 `bugprone-throwing-static-initialization`<br/>
 `bugprone-command-processor`<br/>
+`modernize-use-nodiscard`<br/>
 
 To be evaluated (need to remove exclusion).
 
@@ -150,7 +151,6 @@ To be evaluated (need to remove exclusion).
 To be evaluated (need to enable explicitly).
 
 `modernize-type-traits`<br/>
-`modernize-use-nodiscard`<br/>
 `modernize-use-scoped-lock`<br/>
 
 These apply to codebases which use later standards then C++11 (C++17 is used when building with Qt6) so we cannot simply apply them.
@@ -163,6 +163,11 @@ We are not interested in this.
 
 Reports false positives - see https://github.com/llvm/llvm-project/issues/164125.
 
+`modernize-avoid-bind`<br/>
+`modernize-make-unique`<br/>
+
+Requires us to move to a higher C++ standard by default.
+
 ### Disabled for performance reasons
 
 `portability-std-allocator-const`<br/>

cmake/cxx11.cmake

@@ -9,6 +9,6 @@ macro(use_cxx11)
     endif()
     set(CMAKE_CXX_STANDARD 14 CACHE STRING "C++ standard to use")
   else()
-    set(CMAKE_CXX_STANDARD 11 CACHE STRING "C++ standard to use")
+    set(CMAKE_CXX_STANDARD 17 CACHE STRING "C++ standard to use")
   endif()
 endmacro()