<div class="gmail_quote">2009/10/19 Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On 19.10.09 23:06:05, Morten Volden wrote:<br>
> Hello<br>
><br>
> In order to start somewhere to get acquainted with the code in<br>
> kdevelop/kdeveplatform I decided to have a look at some of the warnings<br>
> generated by the code.<br>
><br>
> The attached patch removes a number of warnings from plugins/subversion<br>
> related to a superfluous const on function return type.<br>
<br>
</div>I guess I should either disable those warnings on the cmake level or<br>
simply declare the code as "ours" by now.<br>
<br>
All the code in kdevsvncpp is copied from rapidsvn to be able to make<br>
certain adjustments but keep the overall codebase (to make our svn<br>
plugins code easier and not cluttered with tons of low-level-C-stuff).<br>
As I do plan to update this codebase every once in a while I'd like to<br>
keep the differences between the original rapidsvn code I've used as<br>
base and what we have in the repository as small as possible.<br>
<br>
So loooong story short: Only functional changes required for features in<br>
the svn plugin are allowed under subversion/kdevsvncpp. Sorry for<br>
wasting your time, any suggestions where I should've written the above<br>
for you to see before doing the changes? (so the next one doesn't do<br>
the same again, as you're the second person already).<br>
<br>
Andreas<br>
<br>
--<br>
You will stop at nothing to reach your objective, but only because your<br>
brakes are defective.<br>
<font color="#888888"><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>
</font></blockquote></div><br><br>Actually I did notice (from the copyright statement) that this particular part of the code was from that project - so I already sent a patch to Rapidsvn which has been included in their trunk. <br>
<br>I wasn't clear to me whether or not the kdevelop codebase was kept in sync with rapidsvn or not (My fault for not asking :-) ).  Sorry for the noise.<br><br>/Morten<br><br>-- <br>- When the split is pulled, mr. Grenade is no longer our friend<br>