In older versions of Alberta, it tries to fix the issues around not having
proper bool in C by redefining bool as _Bool or vice versa, depending
on the language that it is compiled with (C or C++). Clang's stdbool header
does essentially the same and we end up with `typedef bool bool` which is a
compile error. This whole bool/_Bool logic is inside an ifdef in
Albert that is active if HAVE_STDBOOL_H is not defined, in which case
it just includes stdbool.h and uses that. So we can check during
CMake config if that header is available and set the flag accordingly.

Set up default include directories on module target

See merge request core/dune-grid!763

[ug] remove definition of D_SMALL

See merge request core/dune-grid!766

The defintion fixes a missing include within dune-uggrid.

Set the correct FieldInfo size in vtktest

Closes #191

See merge request core/dune-grid!765

[UGGrid] Use std::size_t as loop counter

See merge request core/dune-grid!764

Recently, dune-uggrid has started to replace C arrays by FieldVectors
for storing point positions.  Therefore copying such positions
in dune-grid can now use the assignment operator of FieldVector
instead of hand-coded loops.  This makes the code shorter.  It also
saves a few unnecessary calls to UGGridRenumberer.

This fixes a few signed/unsigned warnings

[uggrid] Simplify domain creation

See merge request core/dune-grid!761

[bugfix][HierarchicSearch] private --> protected.

See merge request core/dune-grid!762

Recent versions of CreateBoundaryValueProblem simply allocated
an object of type STD_BVP (aka std_BoundaryValueProblem).

Therefore, don't call the method, and simply call 'new' instead.

Rather than by name.

All it does is a single assignment.  Instead of calling the method,
do the assignment directly.

This is one further reduction of the complexity of UGGrid setup.

It is not used in dune-uggrid anymore.

'void' is part of an old abstract interface, which only creates
unnecessary complexity for us.

Remove nobindings ci jobs since already in the default set

See merge request core/dune-grid!758

Use less allocations on UGGrid intersection geometries

See merge request core/dune-grid!753

Avoid storing coordinates on the heap by exploiting the fact that we only have
cubes and simplices up to 3D. When the number of corners is the same use an std::array, otherwise use a ReservedVector. This avoid many allocations during grid iteration.

The common case of an intersection is when it has only one face. For this case we use a variant that either holds one face, a vector of faces, or none. This way the common case is stored in the stack whereas the general case is still possible but needs dynamic allocation

There is no inherent need to store geometries in the heap. This changes the data layout to store them in the stack. Incidentally, this also moves coordinates into the geometries to avoid an extra allocation on the vector of coordinates.

replace std::result_of with std::invoke_result to make std20 compilation work

See merge request core/dune-grid!757

Is available since std17

Simplify UGGrid domain creation

Closes #188

See merge request core/dune-grid!755

Now that STD_BVP_Configure has a domain* argument, ADL is enough
to find the correct method in dune-uggrid, and the wrapper in
ugwrapper.hh is not needed anymore.

These two methods have been removed from dune-uggrid.  Instead
'domain' objects can be simply handled via new/delete.

The current code looks like it leaks to the 'domain' object,
but it probably did so before, too.  To be fixed in a
later commit.

Previously, the ConfigPtr variable stored a pointer to that method.
We don't use the extra flexibility offered by that, so let's simplify
the code and just call STD_BVP_Configure directly.

Include std headers

See merge request core/dune-grid!752

Speed up `UGGridGeometry`

See merge request !750