<div><div class="gmail_quote">On Tue, Dec 14, 2010 at 11:54 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;">
Hey all,<br>
<br>
I wonder whether the ProjectBaseItem api is good the way it is. I esp. wonder<br>
whether it should be allowed to setText().<br>
<br>
Personally I think that this is nowhere used and that only setUrl() is used<br>
(or the CTor arguments for the name).<br>
<br>
Furthermore there is already code (esp. in CMakeManager) that expects text()<br>
== baseName().<br>
<br>
I've just added a caching baseName() for speedup<br>
(<a href="http://rpi.edu/~doriad/callgrind.tar.bz2" target="_blank">http://rpi.edu/~doriad/callgrind.tar.bz2</a>) but wonder whether that is not<br>
maybe overkill. I'd like to remove it again and just reuse text() instead.<br>
<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></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8">Makes sense to me. A patch with the proposed changes would be nice :)<div><br></div><div>Aleix<br></div></div>