<div class="gmail_quote">On Mon, Oct 11, 2010 at 11:43 AM, 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;">
Olivier JG, 10.10.2010:<br>
<div><div></div><div class="h5">>   Attached are patches for kdevelop/kdevplatform to change the project<br>
> manager api to be able to remove multiple files at once, and supporting<br>
> code for CMakeManager, CustomMakeManager and GenericManager.<br>
><br>
> Known Issues:<br>
> (1) When removing a folder the CMakeManager still doesn't check if any<br>
> of its subfiles are part of a target (this is current behavior, just not<br>
> fixed with this yet)<br>
> (2) Sometimes after deleting files that aren't part of the buildsystem,<br>
> they will remain in the treeview (I'm not sure if this is current<br>
> behavior, though I think it is as I doubt I added it, not sure if it's<br>
> best to remove the specific row after deletion, or just reload the<br>
> folder, or if this is supposed to happen via some signal and isn't<br>
> working for some reason)<br>
><br>
> I believe I can push, but since this is an API change and a rather large<br>
> patch, posting it here to be safe.<br>
> Let me know if something should be changed/added/removed.<br>
><br>
> Once this is in I hope to deal with Known Issue 1.<br>
<br>
</div></div>generally looks fine and Aleix and me both wanted this change.<br>
<br>
Review:<br>
<br>
1) QWidget* window(QApplication:.activeWindow()) - not required, the<br>
activewindow is a QWidget.<br>
<br>
2) Cmake might still have a problem with this, afaik it reloads the project as<br>
soon as a CMakeLists.txt is removed, hence the folders might be invalid<br>
afterwards. Aleix? Maybe we need to pass one big list of items instead of<br>
differentiating between files and folders?<br></blockquote><div>I'm fine with having files and folders.</div><div>cmake support controls when a reload is inhibited, maybe we could schedule more reloads for afterwards :P</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
3) the typedefs in ibuildsystemmanager seem to be unused?<br>
<br>
4) Aleix will have to review the CMake part<br></blockquote><div>I will, when I find the time.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
So, I'm all in favor of pushing this. to fix (1) you should make sure you don't<br>
pass something like this to the manager:<br>
<br>
foo/<br>
foo/bar/<br>
foo/bar/asdf.txt<br></blockquote><div> </div><div>What do you mean by that? control the UI?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
But instead only:<br>
<br>
foo/<br>
<br>
Bye<br>
<font color="#888888"><br>
--<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>