[Dune] better DEPRECATE

[Christian Engwer]

Hi,

there were two ideas to improve the messages:

a) DEPRECATED_WILL_VANISH_IN(VERSION)
b) DEPRECATED_USE_INSTEAD("some text")

[there names are just to make to point]

I would add the appropriate tests and macros, if someone else provides
me with the GCC magic.

We can still discuss which combinations of a & b we want. The old
macro has to stay, as the new macro can't be constructed backwards
compatible.

Christian

On Sun, Nov 28, 2010 at 02:17:37PM +0100, Andreas Dedner wrote:
> Hi,
> 
> I looked at the dune-fem idea and it not really working as far as I could
> make out. So at the moment I have no suggestion for a replacement.
> 
> Which means, we should include in the deprecated method/class...
> documentation
> 1) what to replace it with
> 2) which is the last release it was not deprecated (perhaps easier
> than guessing the
>      first (future) release in which it will be deprecated).
> 
> The deprecation warning provides file and line number, so looking up
> the documentation
> in the file is not that difficult.
> 
> Until we cook up something better
> Andreas
> 
> On 11/28/2010 10:36 AM, Oliver Sander wrote:
> >Hi Andreas!
> >At the meeting we decided that the DUNE_DEPRECATED macro should be
> >improved
> >to include an actual text message, possibly using existing
> >technology from dune-fem.
> >Maybe you, as a dune-fem expert, could take care of that?
> >
> >best,
> >Oliver
> >
> >Am 28.11.2010 00:24, schrieb Andreas Dedner:
> >>Hi.
> >>We should really try to document what to replace deprecated method with:
> >>That is really important for the users I think.
> >>GeometryType::basicType() is for example used in the virtualrefinement
> >>and I do not really know what to replace it with.
> >>
> >>Andreas
> >>
> >>PS: Compiler output
> >>/grid/common/virtualrefinement.cc:489:105: warning:
> >>‘Dune::GeometryType::BasicType Dune::GeometryType::basicType()
> >>const’ is deprecated (declared at /home/maths/masjal/DUNE/dune-common/dune/common/geometrytype.hh:246)
> >>There are two more places in dune-grid:
> >>./common/test/test-genericquadrature.cc:28: switch (t.basicType())
> >>./common/test/test-quadrature.cc:25: switch (t.basicType())
> >>
> >>
> >>
> >>_______________________________________________
> >>Dune mailing list
> >>Dune at dune-project.org
> >>http://lists.dune-project.org/mailman/listinfo/dune
> >
> >
> 
> 
> _______________________________________________
> Dune mailing list
> Dune at dune-project.org
> http://lists.dune-project.org/mailman/listinfo/dune
>