Bug 3849 – GW: Add T *QPtrDict::operator()()

• Component: New Bugs
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

Add the very strange T *QPtrDict::operator()() which is used in kdom:

T *operator()() { T *ret = (T *)impl.current(); ++impl; return ret; }

And is described here:
http://doc.trolltech.com/3.3/qptrdictiterator.html#operator()

[Eric Seidel (no email)]

Created attachment 2787 [details]
self explanitory patch.

[Maciej Stachowiak]

Comment on attachment 2787 [details]
self explanitory patch.

Eric and I discussed this and agreed it is probably a bad idea to iterate over
a dictionary this way, instead of using an iterator, so better to fix the
caller.

[Maciej Stachowiak]

Comment on attachment 2787 [details]
self explanitory patch.

ok, this is an iterator method, nonstandard, but not so bad.

[Maciej Stachowiak]

Comment on attachment 2787 [details]
self explanitory patch.

I'd rather see use of this syntax removed than have support for it added to
QDict. But re-request review if it turns out to be too much hassle.

[Eric Seidel (no email)]

Comment on attachment 2787 [details]
self explanitory patch.

After further discussions on IRC, I think we came to the conclusion that there
is a real need for this... even if it is a little "odd".