[Dune] [#955] typedef Jacobian in the Geometry class.

[Dune]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

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

User who did this - Robert Klöfkorn (robertk) 

Attached to Project - Dune
Summary - typedef Jacobian in the Geometry class.
Task Type - Feature Request
Category - Grid
Status - New
Assigned To - 
Operating System - Unspecified / All
Severity - Low
Priority - Normal
Reported Version - SVN (pre2.2)
Due in Version - Undecided
Due Date - Undecided
Details - I suggest to change the following two lines in the Geometry interface implementation: 

--- geometry.hh (revision 7670)
+++ geometry.hh (working copy)
@@ -91,12 +91,12 @@
   typedef FieldVector< ctype, cdim > GlobalCoordinate;
   
   //! type of jacobian (also of jacobian inverse transposed)
-  typedef FieldMatrix<ctype,cdim,mydim> Jacobian;
-  //typedef typename ImplementationType :: Jacobian   Jacobian;
+  //typedef FieldMatrix<ctype,cdim,mydim> Jacobian;
+  typedef typename ImplementationType :: Jacobian   Jacobian;
 
   //! type of jacobian transposed
-  typedef FieldMatrix< ctype, mydim, cdim > JacobianTransposed;
-  //typedef typename ImplementationType :: JacobianTransposed  JacobianTransposed;
+  //typedef FieldMatrix< ctype, mydim, cdim > JacobianTransposed;
+  typedef typename ImplementationType :: JacobianTransposed  JacobianTransposed;
 
This means that the implementation of the Jacobian/ JacobianTransposed is obtained from the implementation of the geometry. An already existing typedef in the GeometryDefaultImplementation still uses the FieldMatrix as before. This is a slight change that will simplify the future changes of the geometry interface. Geometry implementations that are not derived from GeometryDefaultImplementation would have to copy the two lines with the typedefs using FiledMatrix to the implementation. 




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

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.