Merge pull request #143 from OpenBrickProtocolFoundation/input-improvements

Improve the inputs

Author: Totto16

.clang-tidy

@@ -1,188 +1,168 @@
 # taken from https://github.com/cpp-linter/cpp-linter-action/blob/main/demo/.clang-tidy
 ---
-Checks: "clang-diagnostic-*,clang-analyzer-*,bugprone-*,misc-*,performance-*,readability-*,portability-*,modernize-*,cppcoreguidelines-*,-modernize-use-trailing-return-type,-readability-named-parameter,-readability-identifier-length,-misc-include-cleaner,-cppcoreguidelines-avoid-magic-numbers,-readability-magic-numbers,-misc-non-private-member-variables-in-classes,-cppcoreguidelines-non-private-member-variables-in-classes,-cppcoreguidelines-special-member-functions"
+Checks: "clang-diagnostic-*,clang-analyzer-*,bugprone-*,misc-*,performance-*,readability-*,portability-*,modernize-*,cppcoreguidelines-*,google-*,llvm-*,cert-*,-modernize-use-trailing-return-type,-bugprone-argument-comment,-misc-include-cleaner,-cppcoreguidelines-avoid-magic-numbers,-readability-magic-numbers,-readability-avoid-nested-conditional-operator,-llvm-namespace-comment,-llvm-header-guard"
 WarningsAsErrors: ""
 HeaderFilterRegex: "oopetris/src/.*"
-AnalyzeTemporaryDtors: false
 FormatStyle: "file"
 CheckOptions:
-    - key: bugprone-argument-comment.CommentBoolLiterals
-      value: "0"
-    - key: bugprone-argument-comment.CommentCharacterLiterals
-      value: "0"
-    - key: bugprone-argument-comment.CommentFloatLiterals
-      value: "0"
-    - key: bugprone-argument-comment.CommentIntegerLiterals
-      value: "0"
-    - key: bugprone-argument-comment.CommentNullPtrs
-      value: "0"
-    - key: bugprone-argument-comment.CommentStringLiterals
-      value: "0"
-    - key: bugprone-argument-comment.CommentUserDefinedLiterals
-      value: "0"
-    - key: bugprone-argument-comment.IgnoreSingleArgument
-      value: "0"
-    - key: bugprone-argument-comment.StrictMode
-      value: "0"
-    - key: bugprone-assert-side-effect.AssertMacros
-      value: assert
-    - key: bugprone-assert-side-effect.CheckFunctionCalls
-      value: "0"
-    - key: bugprone-dangling-handle.HandleClasses
-      value: "std::basic_string_view;std::experimental::basic_string_view"
-    - key: bugprone-dynamic-static-initializers.HeaderFileExtensions
-      value: ",h,hh,hpp,hxx"
-    - key: bugprone-exception-escape.FunctionsThatShouldNotThrow
-      value: ""
-    - key: bugprone-exception-escape.IgnoredExceptions
-      value: ""
-    - key: bugprone-misplaced-widening-cast.CheckImplicitCasts
-      value: "0"
-    - key: bugprone-not-null-terminated-result.WantToUseSafeFunctions
-      value: "1"
-    - key: bugprone-signed-char-misuse.CharTypdefsToIgnore
-      value: ""
-    - key: bugprone-sizeof-expression.WarnOnSizeOfCompareToConstant
-      value: "1"
-    - key: bugprone-sizeof-expression.WarnOnSizeOfConstant
-      value: "1"
-    - key: bugprone-sizeof-expression.WarnOnSizeOfIntegerExpression
-      value: "0"
-    - key: bugprone-sizeof-expression.WarnOnSizeOfThis
-      value: "1"
-    - key: bugprone-string-constructor.LargeLengthThreshold
-      value: "8388608"
-    - key: bugprone-string-constructor.WarnOnLargeLength
-      value: "1"
-    - key: bugprone-suspicious-enum-usage.StrictMode
-      value: "0"
-    - key: bugprone-suspicious-missing-comma.MaxConcatenatedTokens
-      value: "5"
-    - key: bugprone-suspicious-missing-comma.RatioThreshold
-      value: "0.200000"
-    - key: bugprone-suspicious-missing-comma.SizeThreshold
-      value: "5"
-    - key: bugprone-suspicious-string-compare.StringCompareLikeFunctions
-      value: ""
-    - key: bugprone-suspicious-string-compare.WarnOnImplicitComparison
-      value: "1"
-    - key: bugprone-suspicious-string-compare.WarnOnLogicalNotComparison
-      value: "0"
-    - key: bugprone-too-small-loop-variable.MagnitudeBitsUpperLimit
-      value: "16"
-    - key: bugprone-unhandled-self-assignment.WarnOnlyIfThisHasSuspiciousField
-      value: "1"
-    - key: bugprone-unused-return-value.CheckedFunctions
-      value: "::std::async;::std::launder;::std::remove;::std::remove_if;::std::unique;::std::unique_ptr::release;::std::basic_string::empty;::std::vector::empty"
-    - key: cert-dcl16-c.NewSuffixes
-      value: "L;LL;LU;LLU"
-    - key: cert-oop54-cpp.WarnOnlyIfThisHasSuspiciousField
-      value: "0"
-    - key: cppcoreguidelines-explicit-virtual-functions.IgnoreDestructors
-      value: "1"
-    - key: cppcoreguidelines-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
-      value: "1"
-    - key: google-readability-braces-around-statements.ShortStatementLines
-      value: "1"
-    - key: google-readability-function-size.StatementThreshold
-      value: "800"
-    - key: google-readability-namespace-comments.ShortNamespaceLines
-      value: "10"
-    - key: google-readability-namespace-comments.SpacesBeforeComments
-      value: "2"
-    - key: misc-definitions-in-headers.HeaderFileExtensions
-      value: ",h,hh,hpp,hxx"
-    - key: misc-definitions-in-headers.UseHeaderFileExtension
-      value: "1"
-    - key: misc-throw-by-value-catch-by-reference.CheckThrowTemporaries
-      value: "1"
-    - key: misc-unused-parameters.StrictMode
-      value: "0"
-    - key: modernize-loop-convert.MaxCopySize
-      value: "16"
-    - key: modernize-loop-convert.MinConfidence
-      value: reasonable
-    - key: modernize-use-trailing-return-type
-      value: "false"
-    - key: modernize-loop-convert.NamingStyle
-      value: CamelCase
-    - key: modernize-pass-by-value.IncludeStyle
-      value: llvm
-    - key: modernize-replace-auto-ptr.IncludeStyle
-      value: llvm
-    - key: modernize-use-nullptr.NullMacros
-      value: "NULL"
-    - key: performance-faster-string-find.StringLikeClasses
-      value: "std::basic_string"
-    - key: performance-for-range-copy.AllowedTypes
-      value: ""
-    - key: performance-for-range-copy.WarnOnAllAutoCopies
-      value: "0"
-    - key: performance-inefficient-string-concatenation.StrictMode
-      value: "0"
-    - key: performance-inefficient-vector-operation.EnableProto
-      value: "0"
-    - key: performance-inefficient-vector-operation.VectorLikeClasses
-      value: "::std::vector"
-    - key: performance-move-const-arg.CheckTriviallyCopyableMove
-      value: "1"
-    - key: performance-move-constructor-init.IncludeStyle
-      value: llvm
-    - key: performance-no-automatic-move.AllowedTypes
-      value: ""
-    - key: performance-type-promotion-in-math-fn.IncludeStyle
-      value: llvm
-    - key: performance-unnecessary-copy-initialization.AllowedTypes
-      value: ""
-    - key: performance-unnecessary-value-param.AllowedTypes
-      value: ""
-    - key: performance-unnecessary-value-param.IncludeStyle
-      value: llvm
-    - key: readability-braces-around-statements.ShortStatementLines
-      value: "0"
-    - key: readability-else-after-return.WarnOnUnfixable
-      value: "1"
-    - key: readability-function-size.BranchThreshold
-      value: "4294967295"
-    - key: readability-function-size.LineThreshold
-      value: "4294967295"
-    - key: readability-function-size.NestingThreshold
-      value: "4294967295"
-    - key: readability-function-size.ParameterThreshold
-      value: "4294967295"
-    - key: readability-function-size.StatementThreshold
-      value: "800"
-    - key: readability-function-size.VariableThreshold
-      value: "4294967295"
-    - key: readability-identifier-naming.IgnoreFailedSplit
-      value: "0"
-    - key: readability-implicit-bool-conversion.AllowIntegerConditions
-      value: "0"
-    - key: readability-implicit-bool-conversion.AllowPointerConditions
-      value: "0"
-    - key: readability-inconsistent-declaration-parameter-name.IgnoreMacros
-      value: "1"
-    - key: readability-inconsistent-declaration-parameter-name.Strict
-      value: "0"
-    - key: readability-magic-numbers.IgnoredFloatingPointValues
-      value: "1.0;100.0;"
-    - key: readability-magic-numbers.IgnoredIntegerValues
-      value: "1;2;3;4;"
-    - key: readability-redundant-member-init.IgnoreBaseInCopyConstructors
-      value: "0"
-    - key: readability-redundant-smartptr-get.IgnoreMacros
-      value: "1"
-    - key: readability-redundant-string-init.StringNames
-      value: "::std::basic_string"
-    - key: readability-simplify-boolean-expr.ChainedConditionalAssignment
-      value: "0"
-    - key: readability-simplify-boolean-expr.ChainedConditionalReturn
-      value: "0"
-    - key: readability-simplify-subscript-expr.Types
-      value: "::std::basic_string;::std::basic_string_view;::std::vector;::std::array"
-    - key: readability-static-accessed-through-instance.NameSpecifierNestingThreshold
-      value: "3"
-    - key: readability-uppercase-literal-suffix.IgnoreMacros
-      value: "1"
-    - key: readability-uppercase-literal-suffix.NewSuffixes
-      value: ""
+  ## NAMING CONVENTION SECTION
+  - key: readability-identifier-naming.AbstractClassCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.AggressiveDependentMemberLookup
+    value: true
+  - key: readability-identifier-naming.CheckAnonFieldInParent
+    value: false
+  - key: readability-identifier-naming.ClassCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.ClassConstantCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ClassConstantPrefix
+    value: "c_"
+  - key: readability-identifier-naming.ClassMemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ClassMemberPrefix
+    value: "m_"
+  - key: readability-identifier-naming.ClassMethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConceptCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.ConstantCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstantMemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstantParameterCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstantPointerParameterCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstexprFunctionCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstexprMethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ConstexprVariableCase
+    value: "lower_case"
+  - key: readability-identifier-naming.EnumCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.EnumConstantCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.FunctionCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalConstantCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalConstantPointerCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalFunctionCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalFunctionIgnoredRegexp
+    value: "(PrintTo)" ## for gtest
+  - key: readability-identifier-naming.GlobalPointerCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalVariableCase
+    value: "lower_case"
+  - key: readability-identifier-naming.GlobalVariablePrefix
+    value: "g_"
+  - key: readability-identifier-naming.LocalConstantCase
+    value: "lower_case"
+  - key: readability-identifier-naming.LocalConstantPointerCase
+    value: "lower_case"
+  - key: readability-identifier-naming.LocalPointerCase
+    value: "lower_case"
+  - key: readability-identifier-naming.LocalVariableCase
+    value: "lower_case"
+  - key: readability-identifier-naming.MacroDefinitionCase
+    value: "UPPER_CASE"
+  - key: readability-identifier-naming.MemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.MemberPrefix
+    value: "m_"
+  - key: readability-identifier-naming.MethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.NamespaceCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ParameterCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ParameterPackCase
+    value: "lower_case"
+  - key: readability-identifier-naming.PointerParameterCase
+    value: "lower_case"
+  - key: readability-identifier-naming.PrivateMemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.PrivateMemberPrefix
+    value: "m_"
+  - key: readability-identifier-naming.PrivateMethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ProtectedMemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ProtectedMemberPrefix
+    value: "m_"
+  - key: readability-identifier-naming.ProtectedMethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.PublicMemberCase
+    value: "lower_case"
+  - key: readability-identifier-naming.PublicMemberPrefix
+    value: "" # NO PREFIX
+  - key: readability-identifier-naming.PublicMethodCase
+    value: "lower_case"
+  - key: readability-identifier-naming.ScopedEnumConstantCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.StaticConstantCase
+    value: "lower_case"
+  - key: readability-identifier-naming.StaticConstantPrefix
+    value: "s_"
+  - key: readability-identifier-naming.StaticVariableCase
+    value: "lower_case"
+  - key: readability-identifier-naming.StaticVariablePrefix
+    value: "s_"
+  - key: readability-identifier-naming.StructCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.TemplateParameterCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.TemplateTemplateParameterCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.TypeAliasCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.TypedefCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.TypeTemplateParameterCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.UnionCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.ValueTemplateParameterCase
+    value: "CamelCase"
+  - key: readability-identifier-naming.VariableCase
+    value: "lower_case"
+  - key: readability-identifier-naming.VirtualMethodCase
+    value: "lower_case"
+
+    # some needed settings, that are non default
+  - key: bugprone-misplaced-widening-cast.CheckImplicitCasts
+    value: true
+  - key: misc-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
+    value: true
+  - key: bugprone-suspicious-enum-usage.StrictMode
+    value: true
+  - key: bugprone-suspicious-string-compare.WarnOnLogicalNotComparison
+    value: true
+  - key: bugprone-unhandled-self-assignment.WarnOnlyIfThisHasSuspiciousField
+    value: true
+  - key: misc-unused-parameters.StrictMode
+    value: true
+  - key: performance-inefficient-string-concatenation.StrictMode
+    value: true
+  - key: readability-inconsistent-declaration-parameter-name.Strict
+    value: true
+
+  ## special things, that have special values
+  - key: readability-identifier-length.IgnoredVariableNames
+    value: ""
+  - key: readability-identifier-length.IgnoredParameterNames
+    value: "^(os)$" ## std::ostream
+  - key: readability-identifier-length.IgnoredExceptionVariableNames
+    value: ""
+  - key: readability-identifier-length.MinimumLoopCounterNameLength
+    value: 1
+  - key: readability-identifier-length.MinimumExceptionNameLength
+    value: 5
+  - key: readability-function-cognitive-complexity.Threshold
+    value: 50
+  - key: bugprone-assert-side-effect.AssertMacros
+    value: "assert"

meson.build

@@ -7,7 +7,7 @@ project(
         'buildtype': 'debug',
         'optimization': '3',
         'strip': true,
-        'cpp_std': ['c++23', 'c++latest', 'vc++latest', 'c++20'],
+        'cpp_std': ['c++26', 'c++23', 'c++latest', 'vc++latest'],
         'b_ndebug': 'if-release',
 
     },

platforms/build-3ds.sh

@@ -31,7 +31,7 @@ export CMAKE="$BIN_DIR/$TOOL_PREFIX-cmake"
 export PATH="$BIN_DIR:$PATH"
 
 export CC="$COMPILER_BIN/$TOOL_PREFIX-gcc"
-export CXX="$COMPILER_BIN/$TOOL_PREFIX-g"++
+export CXX="$COMPILER_BIN/$TOOL_PREFIX-g++"
 export AS="$COMPILER_BIN/$TOOL_PREFIX-as"
 export AR="$COMPILER_BIN/$TOOL_PREFIX-gcc-ar"
 export RANLIB="$COMPILER_BIN/$TOOL_PREFIX-gcc-ranlib"
@@ -73,6 +73,8 @@ devkitpro = '$DEVKITPRO'
 [binaries]
 c = '$CC'
 cpp = '$CXX'
+c_ld = 'bfd'
+cpp_ld = 'bfd'
 ar      = '$AR'
 as      = '$AS'
 ranlib  = '$RANLIB'