Rather than as a std::unique_ptr.  The unique_ptr was introduced
because originally it was not possible to construct a
LocalGeodesicFEFunction without also binding it.  As this has now
changed we can get rid of the extra allocation required by
std::unique_ptr.

As one further step towards making LocalInterpolationRules
more like (local) functions, introduce a new method 'bind'
accepting a local finite element and a set of (local)
coefficients.  This method replaces the constructor
taking the same arguments. Instead, the local function object
now has to be default constructed.  This is just an
intermediate step, however.

Rather than a LocalFiniteElement.  This makes LocalGeodesicFEFunction
more like a dune-functions-style local function, which I think
it should be.

Rather than a LocalFiniteElement.  This makes LocalGeodesicFEFunction
more like a dune-functions-style local function, which I think
it should be.

I don't think it is worth keeping it, even if it may be
a bit faster than the general code.

The class LocalGfeTestFunctionBasis was removed in a recent commit,
and I forgot to remove a friend declaration for it.

This patch contains only whitespace changes.

Previously, some code was in Dune::GFE, some was in Dune::,
and a lot of it was in the global namespace.

I wrote them years ago mainly because I thought they should exist.
But they were never actually used them for anything, and I don't
really know were they could be used either.  Supposedly they
could be useful for implementing problems without a minimization
formulation, but that's not currently planned, and I don't know
what the precise requirements would be, either.

Since the code is short and almost trivial, and easily resurrected,
this commit removes the two files.  For newcomers this will mean
two less files to wonder about.