A small steps towards its complete removal.  Without the deprecated
methods it is easier to see the code that is still used.

Just to increase the pressure towards its complete removal.

test-parallel-ug: return 77 when run in parallel with sequential dune-uggrid

See merge request !159

Remove unused parameter 'res' from CreateBoundarySegment

See merge request !160

Remove implementation of removed `count<codim>()` template method

See merge request !158

Ansgar Burchardt authored 8 years ago and Christoph Grüninger committed 8 years ago

The method was removed from the `Entity` facade class in
0cfc8f31

gridinfo.hh: Use `subEntities(codim)` method instead of removed `count<codim>()` template method

See merge request !157

The `count<codim>()` template method was removed from the `Entity` facade class in
0cfc8f31

Cleanup UGGrid code

See merge request !156

The implementation is identical for both level and leaf intersections.

The `WorldVector` and `FaceVector` typedefs are already used in some
method declarations and in the implementation.  This patch uses them
consistently everywhere.

The method signature of CreateBoundarySegment has change in dune-uggrid.
The 'res' parameter was not actually used anywhere.  Therefore it has
been removed.

[bugfix] avoid subscript operator on JacobianTransposed

See merge request !154

A geometry's JacobianTransposed implementation may be any type modelling a
linear mapping, i.e., implementing the mv-type methods. Some grids
(e.g., SPGrid) only implement this interface for performance reasons.
However, we require that a JacobianTransposed can be assigned to a
FieldMatrix.

This patch fixes "checkCodim1Mapping" such that it will cast the
JacobianTranposed down to a FieldMatrix before accessing the subscript
operator (although the same result could be achieved using "mv").

UGGrid: append to-be-visited sons at the end of the list

Closes #53

See merge request !150

Handle unused boundary nodes correctly

Closes #55

See merge request !152

For "unused" boundary nodes the special value `-1` is used.  Mapping
this through `isBoundaryNode[-1]` would result in an out-of-bounds
memory access and invalid values in the `vertices_c_style` array.

Closes: #55

Based on the test case provided by Lasse Hinrichsen.

Prepending them to the beginning of the list would mean they are never
visited as the iterator only moves forward.  This required switching to
a `std::list` as the `std::forward_list` does not provide methods to add
items at the end.

This regression was introduced in 20b821be
which replaced `Dune::SLList` with `std::forward_list`, but changed
`push_back` to `push_front` as well.

Closes: #53

If the code path should never be reached, having an exception thrown
makes it easier to nitce it was reached after all.