Switch to CMake build, new logging implementation (#17)

* Switch to CMake
* Logging to Windows event log
* ClassFactory now supports COM aggregation
* C++20
* Improved formatters (wchar_t, formatting of COM, PropVariant/VARIANT objects and string-like objects)
* Replaced most logging and exception macros with template code
* Unity build

Author: mbeckh

.clang-format

@@ -5,29 +5,42 @@ AllowShortFunctionsOnASingleLine: None
 AllowShortIfStatementsOnASingleLine: false
 AllowShortLoopsOnASingleLine: false
 BreakBeforeBinaryOperators: NonAssignment
+BreakBeforeConceptDeclarations: true
 BreakConstructorInitializers: BeforeComma
 BreakInheritanceList: BeforeComma
 ColumnLimit: 0
+DerivePointerAlignment: false
 IncludeBlocks: Regroup
 IncludeCategories: 
-  - Regex:    '^"m3c/.*\.h"'
+  - Regex:    '^"m3c/.*\.h"$'
     Priority: 2
-  - Regex:    '^<m3c/.*\.h>'
+  - Regex:    '^<m3c/.*\.h>$'
     Priority: 2
-  - Regex:    '^<[[:alnum:]]+/.*\.h[hpx]{0,2}>'
+  - Regex:    '^<m4t/.*\.h>$'
     Priority: 3
-  - Regex:    '^<.*\.h[hpx]{0,2}>'
-    Priority: 4
-  - Regex:    '^<.*>'
+  - Regex:    '^<[[:alnum:]]+/.*\.h[hpx]{0,2}>$'
     Priority: 5
+  - Regex:    '^<windows.h>$'
+    Priority: 6
+    SortPriority: 6
+  - Regex:    '^<.*\.h[hpx]{0,2}>$'
+    Priority: 6
+    SortPriority: 7
+  - Regex:    '^<.*>$'
+    Priority: 8
+  - Regex:    '^".*\.events\.h"$'
+    Priority: 4
   - Regex:    '.*'
     Priority: 1
-IncludeIsMainRegex: '(_Test)?$'
+IncludeIsMainRegex: '(\.test\.)?$'
 IndentCaseLabels: false
+IndentRequires: false
 IndentWidth: 4
 MaxEmptyLinesToKeep: 2
+PointerAlignment: Left
+ShortNamespaceLines: 0
 SpaceAfterCStyleCast: true
 SpacesBeforeTrailingComments: 2
-Standard: Cpp11
+Standard: c++20
 TabWidth: 4
-UseTab: ForContinuationAndIndentation
+UseTab: ForIndentation

.clang-tidy

@@ -1,173 +1,197 @@
-Checks: '
+Checks: '-*,
   bugprone-*,
-  cert-*,
+    -bugprone-bad-signal-to-kill-thread,
+    -bugprone-no-escape,
+    -bugprone-posix-return,
+    -bugprone-signal-handler,
+  cert-non-aliased-only-*,
+    cert-dcl21-cpp,
+    cert-dcl50-cpp,
+    cert-dcl58-cpp,
+    -cert-env33-c,
+    cert-err34-c,
+    cert-err52-cpp,
+    cert-err58-cpp,
+    cert-err60-cpp,
+    -cert-exp42-c,
+    cert-flp30-c,
+    -cert-flp37-c,
+    cert-mem57-cpp,
+    cert-msc50-cpp,
+    cert-msc51-cpp,
+    cert-oop57-cpp,
+    cert-oop58-cpp,
   clang-analyzer-*,
   clang-diagnostic-*,
     -clang-diagnostic-gnu-zero-variadic-macro-arguments,
     -clang-diagnostic-language-extension-token,
-  cppcoreguidelines-*,
-    -cppcoreguidelines-avoid-c-arrays,
+  concurrency-*,
+    -concurrency-thread-canceltype-asynchronous,
+  cppcoreguidelines-non-aliased-only-*,
+    cppcoreguidelines-avoid-goto,
+    cppcoreguidelines-avoid-non-const-global-variables,
+    cppcoreguidelines-init-variables,
+    cppcoreguidelines-interfaces-global-init,
+    -cppcoreguidelines-macro-usage,
+    cppcoreguidelines-narrowing-conversions,
+    cppcoreguidelines-no-malloc,
     -cppcoreguidelines-owning-memory,
-    -cppcoreguidelines-pro-type-reinterpret-cast,
-    -cppcoreguidelines-pro-type-union-access,
+    cppcoreguidelines-prefer-member-initializer,
     -cppcoreguidelines-pro-bounds-*,
-  google-*,
-    -google-readability-*,
-    -google-runtime-*,
-  hicpp-avoid-goto,
-  hicpp-exception-baseclass,
-  hicpp-multiway-paths-covered,
-  -hicpp-no-assembler,
-  hicpp-signed-bitwise,
-  llvm-*,
-    -llvm-header-guard,
-    -llvm-qualified-auto,
+    -cppcoreguidelines-pro-type-*,
+      cppcoreguidelines-pro-type-const-cast,
+      cppcoreguidelines-pro-type-cstyle-cast,
+      cppcoreguidelines-pro-type-member-init,
+      cppcoreguidelines-pro-type-static-cast-downcast,
+      cppcoreguidelines-pro-type-vararg,
+    cppcoreguidelines-slicing,
+    cppcoreguidelines-special-member-functions,
+    cppcoreguidelines-virtual-class-destructor,
+  google-non-aliased-only-*,
+    google-build-explicit-make-pair,
+    google-build-namespaces,
+    google-build-using-namespace,
+    google-default-arguments,
+    google-explicit-constructor,
+    google-global-names-in-headers,
+    google-upgrade-googletest-case,
+  hicpp-non-aliased-only-*,
+    hicpp-exception-baseclass,
+    hicpp-multiway-paths-covered,
+    hicpp-signed-bitwise,
+  llvm-namespace-comment,
   misc-*,
+    -misc-misplaced-const,
+    -misc-no-recursion,
   modernize-*,
     -modernize-avoid-c-arrays,
+    -modernize-return-braced-init-list,
     -modernize-use-auto,
     -modernize-use-trailing-return-type,
   performance-*,
   readability-*,
+    -readability-function-cognitive-complexity,
     -readability-function-size,
-    -readability-qualified-auto,
+    -readability-identifier-length,
     -readability-redundant-access-specifiers'
 WarningsAsErrors: ''
-HeaderFilterRegex: 'm3c[\/\\](src|include)[\/\\].*'
+HeaderFilterRegex: '.*'
 ExtraArgsBefore: [
-  '--system-header-prefix=lib/', '-xc++', '-std=c++20', 
-  '-WCL4', '-Wpedantic', '-Wthread-safety', '-Wmicrosoft', '-Wdocumentation', '-Wdocumentation-pedantic', '-Wtautological-compare',
   '-fms-extensions', '-fms-compatibility', '-D_CRT_USE_BUILTIN_OFFSETOF',
-  '-fno-delayed-template-parsing', '-fcomment-block-commands=internal,hideinitializer',
-  '-Im3c/include', '-Ilib/fmt/include', '-Ilib/llamalog/include',
-  '-DUNICODE', '-D_UNICODE', '-DNOMINMAX', '-DWIN32_LEAN_AND_MEAN'
+  '-WCL4', '-Wpedantic', '-Wthread-safety', '-Wmicrosoft', '-Wdocumentation', '-Wdocumentation-pedantic',
+  '-fno-delayed-template-parsing', '/clang:-fcomment-block-commands=internal,hideinitializer',
+  '-DWIN32_LEAN_AND_MEAN=1', '-DNOMINMAX=1'
   ]
 CheckOptions:
+  - key:   bugprone-argument-comment.StrictMode
+    value: 1
   - key:   bugprone-assert-side-effect.AssertMacros
     value: 'assert,_assert,_wassert,_ASSERT,_ASSERTE,_ASSERT_EXPR'
   - key:   bugprone-assert-side-effect.CheckFunctionCalls
-    value: '1'
+    value: 1
+  - key:   bugprone-sizeof-expression.WarnOnSizeOfIntegerExpression
+    value: 1
   - key:   bugprone-suspicious-enum-usage.StrictMode
-    value: '1'
+    value: 1
+  - key:   bugprone-suspicious-string-compare.WarnOnLogicalNotComparison
+    value: 1
   - key:   cppcoreguidelines-narrowing-conversions.PedanticMode
-    value: '1'
+    value: 1
   - key:   hicpp-multiway-paths-covered.WarnOnMissingElse
-    value: '1'
+    value: 1
   - key:   hicpp-signed-bitwise.IgnorePositiveIntegerLiterals
-    value: '1'
+    value: 1
+  - key:   misc-throw-by-value-catch-by-reference.CheckThrowTemporaries
+    value: 0
   - key:   misc-unused-parameters.StrictMode
-    value: '1'
-  - key:   modernize-use-override.IgnoreDestructors
-    value: '1'
+    value: 1
   - key:   performance-inefficient-string-concatenation.StrictMode
-    value: '1'
-  - key:   readability-braces-around-statements.ShortStatementLines
-    value: '0'
+    value: 1
+
+  - key:   readability-identifier-naming.NamespaceCase
+    value: lower_case
+
+  - key:   readability-identifier-naming.FunctionCase
+    value: aNy_CasE
+
+  - key:   readability-identifier-naming.VariableCase
+    value: camelBack
+  - key:   readability-identifier-naming.StaticVariableCase
+    value: camelBack
+
+  - key:   readability-identifier-naming.StaticConstantCase
+    value: CamelCase
+  - key:   readability-identifier-naming.StaticConstantPrefix
+    value: 'k'
+  - key:   readability-identifier-naming.ConstexprVariableCase
+    value: CamelCase
+  - key:   readability-identifier-naming.ConstexprVariablePrefix
+    value: 'k'
+  - key:   readability-identifier-naming.ConstexprVariableIgnoredRegexp
+    value: '^is_[A-Za-z0-9_]+_v$'
+  - key:   readability-identifier-naming.GlobalConstantCase
+    value: CamelCase
+  - key:   readability-identifier-naming.GlobalConstantPrefix
+    value: 'k'
+
   - key:   readability-identifier-naming.ClassCase
-    value: 'lower_case'
+    value: aNy_CasE
   - key:   readability-identifier-naming.StructCase
-    value: 'lower_case'
+    value: aNy_CasE
   - key:   readability-identifier-naming.UnionCase
-    value: 'lower_case'
+    value: aNy_CasE
+  - key:   readability-identifier-naming.EnumCase
+    value: CamelCase
+  - key:   readability-identifier-naming.EnumConstantCase
+    value: CamelCase
+  - key:   readability-identifier-naming.EnumConstantPrefix
+    value: 'k'
+  - key:   readability-identifier-naming.ScopedEnumConstantCase
+    value: CamelCase
+  - key:   readability-identifier-naming.ScopedEnumConstantPrefix
+    value: 'k'
   - key:   readability-identifier-naming.TypeAliasCase
-    value: 'lower_case'
+    value: aNy_CasE
   - key:   readability-identifier-naming.TypedefCase
-    value: 'lower_case'
+    value: aNy_CasE
+
   - key:   readability-identifier-naming.MemberCase
-    value: 'camelBack'
+    value: camelBack
   - key:   readability-identifier-naming.MemberPrefix
     value: 'm_'
-  - key:   readability-identifier-naming.ConstantMemberCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.ConstantMemberPrefix
-    value: 'm_'
   - key:   readability-identifier-naming.PublicMemberCase
-    value: 'camelBack'
+    value: camelBack
   - key:   readability-identifier-naming.PublicMemberPrefix
     value: ''
   - key:   readability-identifier-naming.ClassMemberCase
-    value: 'camelBack'
+    value: camelBack
   - key:   readability-identifier-naming.ClassMemberPrefix
-    value: 'm_'
-  - key:   readability-identifier-naming.ClassConstantCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.ClassConstantPrefix
-    value: 'k'
-  - key:   readability-identifier-naming.EnumCase
-    value: 'lower_case'
-  - key:   readability-identifier-naming.EnumConstantCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.EnumConstantPrefix
-    value: 'k'
-  - key:   readability-identifier-naming.FunctionCase
-    value: 'lower_case'
-  - key:   readability-identifier-naming.NamespaceCase
-    value: 'lower_case'
-  - key:   readability-identifier-naming.VariableCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.VariablePrefix
-    value: ''
-  - key:   readability-identifier-naming.ConstantCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.ConstantPrefix
-    value: 'k'
-  - key:   readability-identifier-naming.LocalVariableCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.LocalVariablePrefix
-    value: ''
-  - key:   readability-identifier-naming.LocalConstantCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.LocalConstantPrefix
-    value: ''
-  - key:   readability-identifier-naming.ParameterCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.ParameterPrefix
-    value: ''
-  - key:   readability-identifier-naming.ConstantParameterCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.ConstantParameterPrefix
-    value: ''
-  - key:   readability-identifier-naming.GlobalVariableCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.GlobalVariablePrefix
-    value: 'g_'
-  - key:   readability-identifier-naming.GlobalConstantCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.GlobalConstantPrefix
-    value: 'k'
-  - key:   readability-identifier-naming.StaticVariableCase
-    value: 'camelBack'
-  - key:   readability-identifier-naming.StaticVariablePrefix
-    value: ''
-  - key:   readability-identifier-naming.StaticConstantCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.StaticConstantPrefix
-    value: 'k'
-  - key:   readability-identifier-naming.GlobalPointerCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.GlobalPointerPrefix
-    value: 'g_p'
-  - key:   readability-identifier-naming.GlobalConstantPointerCase
-    value: 'CamelCase'
-  - key:   readability-identifier-naming.GlobalConstantPointerPrefix
-    value: 'kp'
+    value: 's_'
+
   - key:   readability-identifier-naming.TemplateParameterCase
-    value: 'CamelCase'
+    value: CamelCase
   - key:   readability-identifier-naming.ValueTemplateParameterCase
-    value: 'CamelCase'
+    value: CamelCase
   - key:   readability-identifier-naming.ValueTemplateParameterPrefix
     value: 'k'
   - key:   readability-identifier-naming.ParameterPackCase
-    value: 'camelBack'
+    value: camelBack
+  - key:   readability-identifier-naming.ParameterPackSuffix
+    value: 's'
+  - key:   readability-identifier-naming.MacroDefinitionCase
+    value: 'UPPER_CASE'
+
   - key:   readability-implicit-bool-conversion.AllowIntegerConditions
-    value: '1'
+    value: 1
   - key:   readability-implicit-bool-conversion.AllowPointerConditions
-    value: '1'
+    value: 1
   - key:   readability-inconsistent-declaration-parameter-name.Strict
-    value: '1'
+    value: 1
+  - key:   readability-magic-numbers.IgnorePowersOf2IntegerValues
+    value: 1
   - key:   readability-simplify-boolean-expr.ChainedConditionalReturn
-    value: '1'
+    value: 1
   - key:   readability-simplify-boolean-expr.ChainedConditionalAssignment
-    value: '1'
+    value: 1
   - key:   readability-uppercase-literal-suffix.NewSuffixes
-    value: 'L;LL;LU;LLU;u;ul;ull'
+    value: 'L;LL;u;uL;uLL'

.editorconfig

@@ -11,16 +11,23 @@ guidelines=80
 [*.{cpp,h}]
 guidelines=80,120
 
+[{*.cmake,CMakeLists.txt}]
+indent_style=space
+guidelines=80,120
+
+[*.json]
+indent_style=space
+tab_width=2
+
 [*.md]
 indent_style=space
 tab_width=2
 trim_trailing_whitespace=false
 guidelines=120
 
-[*.yml]
-indent_style=space
+[*.{xml,man}]
 tab_width=2
 
-[{.clang-format, .clang-tidy}]
+[{*.yml,.clang-format,.clang-tidy}]
 indent_style=space
 tab_width=2