Add check for unnecessary moves

.clang-tidy

@@ -1,6 +1,6 @@
 ---
 #disabled checks clash with wanted code
-Checks:          '-*,bugprone-suspicious-semicolon,bugprone-unused-raii,cppcoreguidelines-slicing,cppcoreguidelines-no-malloc,-cppcoreguidelines-pro-type-member-init,modernize-deprecated-headers,modernize-loop-convert,modernize-make-unique,modernize-pass-by-value,modernize-raw-string-literal,modernize-use-auto,modernize-use-bool-literals,modernize-use-default-member-init,modernize-use-equals-default,modernize-use-equals-delete,modernize-use-nullptr,modernize-use-override,modernize-use-using,readability-isolate-declaration,readability-redundant-smartptr-get,readability-redundant-string-cstr,readability-non-const-parameter,performance-inefficient-algorithm,performance-trivially-destructible,readability-simplify-boolean-expr'
+Checks:          'clang-diagnostic-*,clang-analyzer-*,-*,bugprone-suspicious-semicolon,bugprone-unused-raii,cppcoreguidelines-slicing,cppcoreguidelines-no-malloc,-cppcoreguidelines-pro-type-member-init,modernize-deprecated-headers,modernize-loop-convert,modernize-make-shared,modernize-make-unique,modernize-pass-by-value,modernize-raw-string-literal,modernize-use-auto,modernize-use-bool-literals,modernize-use-default-member-init,modernize-use-equals-default,modernize-use-equals-delete,modernize-use-nullptr,modernize-use-override,modernize-use-using,readability-isolate-declaration,readability-redundant-smartptr-get,readability-redundant-string-cstr,readability-non-const-parameter,performance-inefficient-algorithm,performance-trivially-destructible,readability-simplify-boolean-expr,performance-move-const-arg,performance-no-automatic-move'
 WarningsAsErrors: ''
 HeaderFilterRegex: '.*\.h$'
 AnalyzeTemporaryDtors: false
@@ -13,10 +13,6 @@ CheckOptions:
     value:           '::free'
   - key:             cppcoreguidelines-no-malloc.Reallocations
     value:           '::realloc'
-  - key:             cppcoreguidelines-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
-    value:           '1'
-  - key:             cppcoreguidelines-pro-type-member-init.IgnoreArrays
-    value:           '0'
   - key:             google-readability-braces-around-statements.ShortStatementLines
     value:           '1'
   - key:             google-readability-function-size.StatementThreshold
@@ -31,6 +27,14 @@ CheckOptions:
     value:           reasonable
   - key:             modernize-loop-convert.NamingStyle
     value:           CamelCase
+  - key:             modernize-make-shared.IgnoreMacros
+    value:           '1'
+  - key:             modernize-make-shared.IncludeStyle
+    value:           '0'
+  - key:             modernize-make-shared.MakeSmartPtrFunction
+    value:           'std::make_shared'
+  - key:             modernize-make-shared.MakeSmartPtrFunctionHeader
+    value:           memory
   - key:             modernize-make-unique.IgnoreMacros
     value:           '1'
   - key:             modernize-make-unique.IncludeStyle
@@ -57,16 +61,12 @@ CheckOptions:
     value:           '1'
   - key:             modernize-use-equals-default.IgnoreMacros
     value:           '1'
-  - key:             modernize-use-equals-delete.IgnoreMacros
-    value:           '1'
   - key:             modernize-use-nullptr.NullMacros
     value:           'NULL'
   - key:             modernize-use-using.IgnoreMacros
     value:           '1'
-  - key:             readability-implicit-bool-conversion.AllowIntegerConditions
+  - key:             performance-move-const-arg.CheckTriviallyCopyableMove
     value:           '1'
-  - key:             readability-implicit-bool-conversion.AllowPointerConditions
-    value:           '0'
   - key:             readability-simplify-boolean-expr.ChainedConditionalAssignment
     value:           '0'
   - key:             readability-simplify-boolean-expr.ChainedConditionalReturn