The fmt library happily accepts std::string without
any conversions.

Include spdlog via BaseLib/Logging.h, which
provides macro replacements for
ERR/DBUG/WARN/INFO...

XmlStnInterface::rapidReadFile

Moving the implemented function of the class reduces
the rapidxml scope to translation unit.

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*

clang-tidy modernize-use-bool-literals check.

Only in ERR/WARN/INFO/DBUG/OGS_FATAL and similar, not in
formatted output like xml files.

1. Changed urls to https which fixed the following error:

xsd/OpenGeoSysGLI.xsd, at line 1, column 66: Document is not a XML schema.
Error XSDError in ..., at line 3, column 194: Loaded schema file is invalid.

If this error ever pops up again try to cut the internet connection! No joke!

2. Local schema files are now copied to bin-directory during CMake configuration and package generatiion (install step).

Clang format run on near-the-header lines.

Happy New Year

See clang-tidy docu https://clang.llvm.org/extra/clang-tidy/checks/readability-simplify-boolean-expr.html

From llvm documentation https://clang.llvm.org/extra/clang-tidy/checks/misc-string-compare.html
"A common mistake is to use the string’s compare method instead of using the equality or inequality operators. The compare method is intended for sorting functions and thus returns a negative number, a positive number or zero depending on the lexicographical relationship between the strings compared. If an equality or inequality check can suffice, that is recommended. This is recommended to avoid the risk of incorrect interpretation of the return value and to simplify the code. The string equality and inequality operators can also be faster than the compare method due to early termination."

Especially on iterator types and 'new' allocations.

Move parts from FileIO to GeoLib/IO and BaseLib/IO.

This is automatic replacement with
ag '(?<!std::)size_t(?!\w)' --ignore-dir=ThirdParty -l | xargs perl -pi.bak -e  's/(?<!std::)size_t(?!\w)/std::size_t/g'

Also remove unused includes.