3849 2005-07-04 01:48:32 -0700 GW: Add T *QPtrDict::operator()() 2005-07-07 00:53:26 -0700 1 1 1 Unclassified WebKit New Bugs 420+ Mac OS X 10.4 RESOLVED FIXED P4 Enhancement --- 3250 1 eric eric oldest_to_newest 13890 0 eric 2005-07-04 01:48:32 -0700 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() 13891 1 2787 eric 2005-07-04 01:49:19 -0700 Created attachment 2787 self explanitory patch. 13929 2 2787 mjs 2005-07-04 18:42:06 -0700 Comment on attachment 2787 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. 13930 3 2787 mjs 2005-07-04 18:47:33 -0700 Comment on attachment 2787 self explanitory patch. ok, this is an iterator method, nonstandard, but not so bad. 13931 4 2787 mjs 2005-07-04 18:49:21 -0700 Comment on attachment 2787 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. 13933 5 2787 eric 2005-07-04 19:06:46 -0700 Comment on attachment 2787 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". 2787 2005-07-04 01:49:19 -0700 2005-07-07 00:39:59 -0700 self explanitory patch. qptrdict.patch text/plain 507 eric SW5kZXg6IC4va3dxL0tXUVB0ckRpY3QuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3ZzL3Jvb3Qv V2ViQ29yZS9rd3EvS1dRUHRyRGljdC5oLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjI4CmRpZmYg LXAgLXUgLXIxLjI4IC4va3dxL0tXUVB0ckRpY3QuaAotLS0gLi9rd3EvS1dRUHRyRGljdC5oCTIw MDQvMTAvMDUgMjA6MjU6MTkJMS4yOAorKysgLi9rd3EvS1dRUHRyRGljdC5oCTIwMDUvMDcvMDQg MDg6MTk6NTUKQEAgLTcyLDYgKzcyLDcgQEAgcHVibGljOgogCiAgICAgVCogdG9GaXJzdCgpIHsg cmV0dXJuIChUKikoaW1wbC50b0ZpcnN0KCkpOyB9CiAKKyAgICBUICpvcGVyYXRvcigpKCkgeyBU ICpyZXQgPSAoVCAqKWltcGwuY3VycmVudCgpOyArK2ltcGw7IHJldHVybiByZXQ7IH0KICAgICBU ICpvcGVyYXRvcisrKCkgeyByZXR1cm4gKFQgKikrK2ltcGw7IH0KIAogcHJpdmF0ZToK