<div class="gmail_quote">On Mon, Jul 12, 2010 at 2:23 PM, Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de">mail@milianw.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On Monday, 12. July 2010 04:46:26 Aleix Pol wrote:<br>
> hi,<br>
> This patch solves the problem we have when we try to refresh the<br>
> projectmodel from another thread, it makes the method call to be triggered<br>
> whenever the model thread runs the event loop. I know it's not a clean<br>
> implementation (that's why I'm sending it here).<br>
><br>
> I'm sending a patch so that you can try how the reload feels after that<br>
> change. I'm not commiting right away because I understand there's drawbacks<br>
> and I suppose you will want to give your opinion. I think we could give it<br>
> a try though.<br>
<br>
</div></div>Shouldn't the check go into appendRow itself? Instead of the ctor of the item?<br></blockquote><div>Well, I put that in the ctor because it's there when you just create the file and don't care anymore. When you do an appendrow you might want to check it after (and of course it will result in a problem, since the addition is delayed). I'm fine with creating a delayedAppendRow or something like that, though.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Does it work with Generic Manager as well?<br></blockquote><div>Haven't tried but it should. It should work the same as long as you add items from the main (model's) thread.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
Have you tried showing the project root item directly upon starting the import<br>
job, which used to show the problems for me back then in cmake manager? Imo<br>
this should work with your patch then?<br></blockquote><div>This should probably work, yes. Haven't tried though.</div><div>And I still think the user will like to have some feedback on the project imported (maybe not being actually capable to work on the project but something).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
And: Even though it looks a bit hackish, I'm all in favor. Simply because the<br>
user impression will get much better.<br></blockquote><div>It is hackish but considering than working from another thread is just broken, I think we should consider using it (or twisting the API a little further).</div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Bye<br>
<font color="#888888">--<br>
Milian Wolff<br>
<a href="mailto:mail@milianw.de">mail@milianw.de</a><br>
<a href="http://milianw.de" target="_blank">http://milianw.de</a><br>
</font><br>--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
<br></blockquote></div><br>