[Dune] [#819] Remove enum FieldVector::size

Dune flyspray at dune-project.org
Mon Sep 13 13:57:08 CEST 2010


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#819 - Remove enum FieldVector::size
User who did this - Martin Nolte (nolte)

----------
I guess the stuff in dune/localfunctions/utility can be changed to 'dimension', but this will be a piece of work that I currently don't intend to do. The use of 'size' over 'dimension' originates from the missing preference in the documentation. The is no clear statement whether to use 'size' or 'dimension'.

I guess that there is lots of other code (e.g., dune-fem), where size is used instead of dimension. As we always want backward-compatibility, I suggest to deprecate the enum instead of removing it (in case we decide for the removal). If an enum cannot be deprecated, simply make it a 'static const int' and deprecate it.

Personally, I don't see the gain in removing the 'size'. It creates a lot of work for very little gain (though we seem to see many such suggestions over the last weeks). But I am in favor of a clearly documented statement which of the two constants is preferred.
----------

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

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