<div class="gmail_quote">On Thu, Oct 14, 2010 at 1:55 AM, Olivier JG <span dir="ltr"><<a href="mailto:olivier.jg@gmail.com">olivier.jg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"> On 10/13/2010 11:38 PM, Aleix Pol wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
API wise:<br>
- You can't call it removeFilesAndFolders and pass ProjectBaseItem's because there might be targets there too (target removal doesn't work with cmake yet, though, but we could add that easy). Maybe removeItems()?<br>

</blockquote></div>
That was my idea originally, but then I noticed that removeTarget is in IBuildSystemManager. IOW, from the API point of view, as it stands, removing files/folders and targets are two different things. I asked Milian and he said to keep it that way and go with removeFIlesAndFolders.<br>

As far as what to do, currently the ProjectManagers get a nice clean list of only folders and files passed to removeFilesAndFolders, and as long as it stays that way, it's not an issue. Perhaps I should ASSERT that stance though?<br>

<br>
Thanks for reviewing<div><div></div><div class="h5"><br>
-Olivier JG<br>
<br>
-- <br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org" target="_blank">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>
</div></div></blockquote></div><br><div>Fine, assert in case a target is passed.</div>