Remove method index<cc> and subIndex<cc> from IndexSet
The index set currently provides the index and subIndex methods twice:
template< class Entity > Index index ( const Entity &e ) const
template< int codim > Index index ( const typename Grid::template Codim< codim >::Entity &e ) const
Of course, both methods do the same thing. The second version simply checks the type of entity. We have better checks for this now, e.g.:
template< class Entity, std::enable_if_t< std::is_same< Entity, typename Grid::template Codim< Entity::codimension >::Entity >::value, int > = 0> Index index ( const Entity &e ) const
I think dropping the second form (maybe in favor of the third) would clarify the interface for the user.