KDE/kdelibs/kdecore

[Andreas Pakulat]

On 08.07.08 10:19:57, Thiago Macieira wrote:
> Roland Harnau wrote:
> >2008/7/8, Thiago Macieira <thiago at kde.org>:
> >> Andreas Pakulat wrote:
> >>>I'm not sure why the const_this is needed here anyway, shouldn't a
> >>> simple const_iterator and ::constEnd() be sufficient too?
> >>
> >> It's needed because of the missing constFind() function.
> >
> >const_iterator QHash::constFind ( const Key & key ) const
> >
> >does exist, at least in Qt's API reference. I think it really improves
> > readability if  the const versions (constBegin(), constEnd(),
> > constFind()) are used without the need to rely on subtle rules for
> > resolving overloaded methods.
> 
> Uh... you're right. It exists for QMap too.

Hmm, seems I need new glasses, I didn't see it in the API docs :) 

I'll defer the change (removing the "strange" const_this) after the
unfreeze as I don't think its important enough to be comitted right now.

Andreas

-- 
Change your thoughts and you change your world.