[Bug libstdc++/51560] New: std::set<T>::cbegin()/cend() etc. return iterator instead of const_iterator

[Bug libstdc++/51560] New: std::set<T>::cbegin()/cend() etc. return iterator instead of const_iterator

[carsten_neumann at gmx dot net]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51560

             Bug #: 51560
           Summary: std::set<T>::cbegin()/cend() etc. return iterator
                    instead of const_iterator
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: carsten_neumann@gmx.net


Created attachment 26096
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26096
Trivial (untested) patch

The return type of the member functions cbegin,cend,crbegin,crend for std::set
and std::multiset are wrong, they should be
const_iterator/const_reverse_iterator respectively.

Attaching the trivial patch (untested).

I don't have copyright assignment, but since the patch is trivial and only
changes a few lines I'm hoping it's not needed.

[Bug libstdc++/51560] std::set<T>::cbegin()/cend() etc. return iterator instead of const_iterator

[redi at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51560

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> 2011-12-15 01:02:32 UTC ---
It's the same type, as permitted by the standard:

"For associative containers where the value type is the same as the key type,
both iterator and const_iterator are constant iterators. It is unspecified
whether or not iterator and const_iterator are the same type."