[Dune] [#827] rbegin of FieldVector and ISTL classes has surprising semantics

Dune flyspray at dune-project.org
Tue Oct 12 10:25:31 CEST 2010


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

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

User who did this - Christian Engwer (christi) 

Attached to Project - Dune
Summary - rbegin of FieldVector and ISTL classes has surprising semantics
Task Type - Bug Report
Category - Dune Core Modules
Status - New
Assigned To - 
Operating System - Unspecified / All
Severity - Low
Priority - Normal
Reported Version - 2.0
Due in Version - Undecided
Due Date - Undecided
Details - The methods rbegin() and rend() of FieldVector, DenseVector, DynamicVector and the classes in ISTL return an Iterator. The usual behaviour of this method would be to return a reverse_iterator. The main difference is, that the iterator has to be decremented using '--', while the reverse_iterator is decremented using '++'.

This behaviour may come as a big surprise to new users. I do understand, the current behaviour is needed in ISTL, thus I do not want to suggest  to remove this feature, but I suggest to rename the methods to avoid additional confusion. Renaming the methods should be fairly simple.

Opinions?

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

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