XMLQtInterface::insertStyleFileDefinition

find *Lib -type f | xargs sed -i 'N;s%\(^\/\*\*\)\n \*$%\1%'

Mostly in the file headers.

find *Lib -type f | xargs sed -i 'N;s%^\s\+\*\n\(\s\+\*/\)%\1%'

Fixes multiple copyright messages in the doxygen docu.

ag '\\file' -L --ignore=ThirdParty *Lib |xargs  sed -i 's/\(.*\)\\copyright/\1\\file\n\1\\copyright/'

initializer-list constructor should not be declared explicit.
Constructors that take a single std::initializer_list
parameter should also omit explicit, in order to support
copy-initialization.

Including
 - usage of static_cast instead of c-style cast,
 - single variable declaration per line,
 - removing const from return types
 - omit unnecessary string to char* conversions,

The used clang-tidy checks are:
google-readability-casting,
readability-const-return-type,
readability-container-size-empty,
readability-delete-null-pointer,
readability-deleted-default,
readability-else-after-return,
readability-function-size,
readability-identifier-naming,
readability-inconsistent-declaration-parameter-name,
readability-isolate-declaration,
readability-misleading-indentation,
readability-misplaced-array-index,
readability-named-parameter,
readability-redundant-*,
readability-simplify-*,
readability-static-accessed-through-instance,
readability-static-definition-in-anonymous-namespace,
readability-string-compare,
readability-uniqueptr-delete-release

Most of the current .clang-tidy contained checks:
google-explicit-constructor
google-readability-braces-around-statements
modernize-loop-convert
modernize-use-default-member-init
modernize-use-emplace
modernize-use-using
readability-braces-around-statements
readability-else-after-return
readability-named-parameter
*readability-namespace-comments
*readability-inconsistent-*
readability-redundant-member-init
readability-simplify-boolean-expr
readability-string-compare
*explicit-conversions*

Dmitri Naumov authored 5 years ago and Tom Fischer committed 5 years ago

This is a variation of getOrError() for containers using a predicate.

This fixes a warning about implicitly deleted copy assignment.
A default copy assignment cannot be created with const
members.

clang-tidy modernize-use-transparent-functors check.

clang-tidy modernize-use-auto check.

clang-tidy modernize-use-default-member-init check

clang-tidy bugprone-macro-parentheses check.

Also those where the second and so forth arguments are defaulted.

The MathLib::TemplateVector3 is an exception, because of the many
required changes through the code, which relies on the implicit
conversion from TemplatePoint<T, 3> to TemplateVector3<T>.

clang-tidy '*explicit*' check.

The enclosed code depends on OS, not compiler.

[[noreturn]] attribute is supported by now.