CheckSubEntityGeometry forces grid implementors to sublass Entity

Is that on purpose? Previously it was possible for grids to use their own implementation without inheriting from Dune::Entity. That seemed a lot more convenient.

Just for one static check for Dune::Capabilities::hasEntity<GI,codim>::v in https://gitlab.dune-project.org/core/dune-grid/blob/908ed8526a553f2b1fbb5f9120e259a0f642daba/dune/grid/test/checkgeometry.hh#L53-L57 all entities have to be subclasses of Dune::Entity. That is quite a bummer.

I even think that this static test is wrong (or better: testing the wrong direction). It tests whether hasEntity<GI,codim>::v is true if that codim exists, but a grid might want to mimic as one without that codim even though it offers functions for it because the implementation might not be complete or buggy.

Mentioned in merge request !113 (closed)

Mentioned in merge request !114 (merged)

Mentioned in issue #14 (closed)

Status changed to closed by commit 67e5c5a7

Mentioned in commit b50e3159

Mentioned in merge request !129 (merged)

Mentioned in commit 67e5c5a7

Mentioned in commit 0b61d119