This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix PR libstdc++/13462 std::set's pointer is not right


Andrew Pinski <pinskia@physics.uc.edu> writes:

| The problem is that std::set's pointer is defined based on
| _Rep_type::const_pointer and not _Rep_type::pointer.
| This patch fixes that and others in libstdc++.
| 
| OK?
|     _Rep_type _M_t;  // red-black tree representing multiset
|   public:
| -  typedef typename _Rep_type::const_pointer pointer;
| +  typedef typename _Rep_type::pointer pointer;

I prefer the change be

   + typedef typename _Alloc::pointer pointer;

To rflect Standard wording.  Same for the rest.

With those changes, the patch is OK.

-- Gaby


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]