#1310 Wrong constructor initializer argument
Metadata
Property | Value |
---|---|
Reported by | Olaf Ippisch (olaf.ippisch@iwr.uni-heidelberg.de) |
Reported at | Jun 1, 2013 06:29 |
Type | Bug Report |
Version | 2.2 |
Operating System | Unspecified / All |
Last edited by | Markus Blatt (markus@dr-blatt.de) |
Last edited at | Jun 4, 2013 08:30 |
Closed by | Markus Blatt (markus@dr-blatt.de) |
Closed at | Jun 4, 2013 08:30 |
Closed in version | Unknown |
Resolution | Fixed |
Comment |
Description
In dune-grid/dune/grid/alugrid/2d/grid_imp.cc there is in a initialisation list a call to geomTypes_(dim+1,1), where geomTypes_ is a vector of vector. The second argument is illegal and leads to a compiler problem with clang as in C++11 this constructor is explicit and the argument is no longer automatically converted. The correct argument would be: std::vector(1)
The full patch is: --- a/dune/grid/alugrid/2d/grid_imp.cc +++ b/dune/grid/alugrid/2d/grid_imp.cc @@ -119,7 +119,7 @@ namespace Dune : mygrid_ (0) , maxLevel_(0) , coarsenMarked_(0) , refineMarked_(0)
-
, geomTypes_(dim+1,1)
-
, geomTypes_(dim+1,std::vector<GeometryType>(1)) , hIndexSet_(*this) , localIdSet_ (*this) , levelIndexVec_( MAXL, (LevelIndexSetImp *) 0 )
I would be happy if anybody with write access could correct this.
This does not affect the trunk, as the respective line is gone there.