[Dune] Notification from Dune

Flyspray dune at hal.iwr.uni-heidelberg.de
Mon Nov 27 22:12:13 CET 2006


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below.

User who did this: - Oliver Sander (sander)

Attached to Project - Dune
Summary - Mapper classes expect redundant interface of layout class
Task Type - Bug Report
Category - Grid
Status - New
Assigned To - Oliver Sander
Operating System - All
Severity - Low
Priority - Normal
Reported Version - SVN
Due in Version - 1.0
Due Date - Undecided
Details - The various mapper classes in dune-grid/grid/common expect a
layout class that tells which entities of a grid carry degrees of
freedom.  These layout classes are supposed to have a method
operator()(int codim, GeometryType type).  The first argument is
superfluous, because the layout class gets the grid dimension as a
template parameter and GeometryType objects know their dimension.  The
redundancy is a leftover from the introduction of the GeometryType,
and should be removed, because it is confusing.

The problem is that I do not see a way to remove the first argument
without inducing compile failures in application code.  These failures
will not be serious, though.  In a few days I will change the mappers
such that they will expect their layout classes to have
operator()(GeometryType type).  Any Dune application that defines its
own layout classes will then fail to compile.  To fix these failures
please remove the first argument of operator() of each layout class
you define, and replace all occurrences of codim with dim-type.dim().

Of course if someone comes up with a smoother way to make this
transition I will be more than happy to comply.

More information can be found at the following URL:
http://hal.iwr.uni-heidelberg.de/flyspray/?do=details&id=210

You are receiving this message because you have requested it from the
Flyspray bugtracking system.  You can be removed from future
notifications by visiting the URL shown above.





More information about the Dune mailing list