[bugfix] include the matrixmarket header to handle default argument

The standard states (8.3.6/4) that adding default arguments in later
function declarations are only valid for non-template functions.
In matrixmarket.hh, the function template loadMatrixMarket is declared
and a default argument for the last parameter is provided. In
owneroverlapcopy.hh, OwnerOverlapCopyCommunication contains a friend
declaration of loadMatrixMarket without the default parameter. If now
owneroverlapcopy.hh is included before matrixmarket.hh, the friend
declaration is the first declaration and the default parameter in
matrixmarket.hh is invalid.
By including matrixmarket.hh in owneroverlapcopy.hh, we make sure that
the declaration which provides the default argument is always the first
one.

(cherry picked from commit 925dbac3)
Signed-off-by: Carsten Gräser <graeser@dune-project.org>

dune/istl/owneroverlapcopy.hh

@@ -31,6 +31,7 @@
 #include "solvercategory.hh"
 #include "istlexception.hh"
 #include <dune/common/parallel/collectivecommunication.hh>
+#include <dune/istl/matrixmarket.hh>
 
 template<int dim, template<class,class> class Comm>
 void testRedistributed(int s);