[Dune] Re: gridype.hh

Andreas Dedner andreas.dedner at mathematik.uni-freiburg.de
Tue Aug 7 16:02:06 CEST 2007

of course we can move gridtype.hh zu utility
I would suggest for the moment to move the header
but keep a link in dgf with a warning since all
our code depends on that file.

But what is the point of removing the includes?
You want to replace dgfalu.hh with alugrid.hh?
and then repeat all the preproc. magic again in
dgfparser.hh to get at dgfalu.hh.
Including all dgf*.hh files in dgfparser.hh does
not seem reasonable to me, since that would mean including
all gridheaders (alu,alberta,yasp.ug...) even if only
one of them is needed.
Leaving the dgf*.hh in gridtype.hh even if it is
in utility would have the advantage that the preproc. magic
can be used with dgf and without.
Best regards 

On Tue, 7 Aug 2007 14:28:39 +0200
 Flyspray <dune at dune-project.de> wrote:
> A new Flyspray task has been opened.  Details are below.
> User who did this: - Christian Engwer (christi)
> Attached to Project - Dune
> Summary - gridtype.hh dows not belong to DGFparser
> Task Type - Bug Report
> Category - Grid
> Status - New
> Assigned To - 
> Operating System - All
> Severity - Low
> Priority - Normal
> Reported Version - SVN
> Due in Version - 
> Due Date - Undecided
> Details - gridtype.hh offers some preprocessor magic to
> construct the
> correct typedef for the grid class. The usage is neither
> limited to,
> nor required by the DGFParser. As we already discussed
> some time ago,
> gridtype.hh should go to grid/utility. The DGF includes
> in gridtype.hh
> should be moved to dgfparser.hh.
> More information can be found at the following URL:
> 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.
> _______________________________________________
> Dune mailing list
> Dune at dune-project.org
> http://lists.dune-project.org/mailman/listinfo/dune

More information about the Dune mailing list