<div class="gmail_quote">On Wed, Oct 13, 2010 at 5:13 PM, 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;">
 Attached are new patches for kdevelop and kdevplatform. Egregious errors can be attributed to time of night.<br>
Now, deleting a folder will have any subfiles removed from targets as needed, and the api is removeFilesAndFolders.<br>
Known issue is that the remove option shouldn't be present when removing target subfiles or the project dir (gasp), this is currently worked around for safety and cleanliness. Once this goes in, I'll take a look at fixing the context menu.<br>
<font color="#888888">
-Olivier JG<br>
<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><div>API wise:</div><div>- 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()?</div>
<div><br></div><div>On KDevelop patch:</div><div>- can you add const & on the parameters? All the rest of the code is using it, we won't change that just for this case</div><div>- instead of passing parameters as class&, pass them as pointers, it's how we usually work with widgets and looks more clear imo.</div>
<div><br></div><div>But this is more like small comments, I think the patches are good globally.</div><div>Aleix</div><div><br></div>