However do not set that option permanently (and also disable RemoveSemicolon by
default), since the clang-format manual states that these options can lead to
incorrect formatting and thus their result should be carefully reviewed.

Also do some coding style manual enforcement.

Closes: #128

It was previously in case-sensitive alphabetical order (i.e. ASCII order).

Ref. #128

– use std::pair instead of std::tuple in internal storage
– make Environment::getFunction() symmetric with Environment::getVariable()

Automatically detected by clang-tidy using
performance-inefficient-vector-operation check.

Automatically detected by clang-tidy with performance-move-const-arg check.

Do not make the modification for Tokenizer::location type, since we have no
guarantee that the type will remain trivially-copyable in the future.

Automatically detected by clang-tidy using performance-for-range-copy check.

Automatically detected by clang-tidy using performance-unnecessary-value-param
check.

There is actually no bug in MinGW. The different behaviour under Windows is
expected, because the implicit conversion there is only to std::wstring.

It would mishandle trailing slashes or backslashes in std::filesystem::path, see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88881

The bug is now fixed in the version of MinGW in Debian “bookworm” 12.

Ref. Madysson/estimation-codes#11

Also add some annotations to remove some false positives.

There remain some boost-related false positives, it’s unclear how to suppress
them.

Automatically detected using clang-tidy with bugprone-reserved-identifier
check.

By the way, homogeneize the define identifiers in relation to camel case
convention.

Automatically detected using clang-tidy with modernize-use-emplace check.

Automatically detected using clang-tidy with modernize-use-nodiscard check.

Adapt the code accordingly.

Macro processor: fix tracking of line numbers after @#define, @#include and @#includepath directives

Closes: #108

This is unsafe since the find() method can return a past-the-end iterator,
which should be tested for.

Replace most instances by calls to the std::map::at() method (which throws if
the key is unknown), and which is incidentally more readable.

By the way, turn those classes into structs since all their members are public.

As a consequence, add many [[maybe_unused]] attributes for legitimate
cases (mainly in virtual functions).

A function defined entirely inside a class/struct/union definition is
implicitly an inline function.

Previously it was set to 1.

Also do the same change for variable defined on the command-line, for
consistency.

Closes: #43

In particular, use this feature in many loops which feature a special treatment
for the first iteration, using a boolean variable (replacing iterator
manipulation). By the way, also use std::exchange() to simultaneously test the
value of this variable and update it.

We follow the advice given by Josuttis in his book about move semantics.

Deleting those member fuctions can be a bug if we want to allow copy semantics,
because overload resolution will no longer fallback to the copy
constructor/assignment operator when given an rvalue.

In particular, this explain why it was not possible to delete the move
assignment operator of the StaticModel class.