<div><div>Hi Matthias and all,<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">As you can see in the KCModuleLoader and KCModuleProxy code there's no event
<br>processing happening between instantiating the KCModule and connecting the<br>changed signal. So if 0 doesn't work and 10/100 works you should really start<br>debugging as something wrong is happening (some code calling processEvents or
<br>so).</blockquote><div><br>Yeah is just the fact that hard coded values makes me nervous... but as the commit says it can be happily reverted if it is confusing. I'm not against renaming the method either.<br></div>
<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Which is the reason why my gut feeling is against the init method (which is a<br>rather common (private) function name, btw, in many classes that have more
<br>than one ctor).</blockquote><div><br>Yep.Probably it sounds like a private method name. We can rename it to something else. It is not a problem, the important thing is what it does. <br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
You're saying load() won't work because it will "emit changed(false)"? But it<br>doesn't have to. For your special case the load() function will figure out<br>that the configuration it shows is different than what's stored in the config
<br>file so it can "emit changed(true)".</blockquote><div><br>No, I'm saying that if we usually call load() from the constructor of the KCModule we are still on the same problem, because that emit changed(true) that is supposed to be on the load() method (what I don't consider is right, because we want that to happen when the plugin is loaded the first time, not other times: note that if a plugin changed and the user clicks "Cancel", as the dialog is "cached", load() is called) is not connected to any slot yet.
<br><br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">In summary: I don't think we have enough justification for init().</blockquote>
<div><br>Right. We could call kcm->load() after the connections are done. What do you think ?<br><br>Could you please revert my patch ? I have never reverted a patch and I've no SVN right now (probably ADSL is coming today...), and I wont be able to be on the other place with ADSL within Monday. That is the reason because I did the commit since I hadn't time to come back on Monday. Sorry.
<br></div></div><br><br>Bye and thank you,<br>Rafael Fernández López.