[Dune] [#849] YaspGrid creates entities of unsupported codimension in communicate() (Attachment added)

Dune flyspray at dune-project.org
Mon Dec 6 12:39:31 CET 2010


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

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

User who did this - Steffen Müthing (smuething) 

Attached to Project - Dune
Summary - YaspGrid creates entities of unsupported codimension in communicate()
Task Type - Bug Report
Category - YaspGrid
Status - Unconfirmed
Assigned To - 
Operating System - Unspecified / All
Severity - Medium
Priority - Normal
Reported Version - SVN (pre2.1)
Due in Version - Undecided
Due Date - Undecided
Details - In YaspGrid, a TMP steps through all codimensions of the grid and repeatedly calls the method
communicateCodim(), which is templated on the codimension. That method then checks
whether or not that codimension is supported (vertices or cells) and either starts the
data transfer or throws an exception. Unfortunately, the compiler is still forced to instantiate
communicateCodim() for all codimensions, which is a waste of compile time. Moreover,
and much more problematic, that method somehow constructs entity types of unsupported
codimensions and passes them to the DataHandle. At that point I have a problem, because
I need to wrap these entities in my meta grid before I can forward them to user code. Trying
to do that (saving them in an EntityPointer) causes a compilation error (which is a good thing IMHO).

The attached patch fixes the problem by making sure that communicateCodim() only gets
instantiated for codim == 0 and codim == dim.

One or more files have been attached.

More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=849

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.




More information about the Dune mailing list