Debug iterator cleanup

Jonathan Wakely jwakely@redhat.com
Mon May 23 11:29:00 GMT 2016


On 22/05/16 17:21 +0200, François Dumont wrote:
>Hi
>
>    I just want to make sure that you agree that I can remove the 
>@todo by implementing operator-> this way.
>
>    * include/debug/safe_iterator.h
>    (_Safe_iterator<>::operator->()): Implement using underlying iterator
>    operator ->.
>    * include/debug/safe_local_iterator.h
>    (_Safe_local_iterator<>::operator->()): Likewise.

We never use _Safe_iterator to wrap a raw pointer, right? Raw pointers
don't have operator-> so that wouldn't work, but I think we only wrap
class types (the containers that might use raw pointers use
__normal_iterator instead).

So it looks good to me, thanks.



More information about the Libstdc++ mailing list