Version 2 of the patch, now with the signal in KPageDialog and KConfigDialog responding to this.<br><br><div><span class="gmail_quote">On 10/22/07, <b class="gmail_sendername">Aaron J. Seigo</b> <<a href="mailto:aseigo@kde.org">
aseigo@kde.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">On Monday 22 October 2007, Joris Guisson wrote:<br>> On 10/22/07, Aaron J. Seigo <
<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>> wrote:<br>> > On Monday 22 October 2007, Joris Guisson wrote:<br>> > > Nobody interested ? I guess my patch must be fine then, shall I commit<br>> > > ?
<br>> ><br>> > having a second QMap seems ugly and a bit wasteful. can you do something<br>> > like:<br>> ><br>> > QMap<QWidget *, KConfigDialogManager *>::iterator it =<br>> > d->
managerForPage.begin();<br>> ><br>> > while (it != d->managerForPage.end()) {<br>> >     if (it.value() == item->widget()) {<br>> >         delete it.value();<br>> >         d->managerForPage.erase
(it);<br>> >     }<br>> >     ++it;<br>> > }<br>><br>> That was my first approach and it failed miserably, you see the widget you<br>> give to KConfigDialog, gets stuffed into a KVBox.<br><br>then i wonder if this would work:
<br><br>if (item->widget()->isAncestorOf(it.value())) {<br><br>> Btw, didn't know binary compatibility was important at this point in time.<br>> I can understand once 4.0 is out, but now ?<br><br>well, we *are* trying to calm down and start respecting the API. tagging on
<br>wednesday and all.</blockquote><div><br>True, but making one method virtual doesn't really change that much, and wouldn't break compiling stuff which uses KPageDialog.<br><br>Anyway, the second version of the patch doesn't do this, so BC should not be broken.
<br> </div>Joris,<br><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">--<br>Aaron J. Seigo<br>humru othro a kohnu se<br>GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
<br><br>KDE core developer sponsored by Trolltech<br><br></blockquote></div><br>