<div class="gmail_quote">On Tue, Aug 4, 2009 at 2:07 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<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 30.07.09 22:52:03, Stefan Valouch wrote:<br>
> On Thursday 30 July 2009 21:35:41 Andreas Pakulat wrote:<br>
> > On 28.07.09 18:19:02, Stefan Valouch wrote:<br>
> > > Hi everyone<br>
> > ><br>
> > > I just succeeded in making the java language plugin from<br>
> > > playground-devtools/kdevelop4-extra-plugins/java compile. Mainly, I<br>
> > > corrected the code to be compiled against trunk API and silent some g++<br>
> > > warnings. The patch is here:<br>
> > > <a href="http://valouch.info/patches/kde/playground-devtools/0001-" target="_blank">http://valouch.info/patches/kde/playground-devtools/0001-</a><br>
> > > Make-the-java-plugin-compile.patch<br>
> ><br>
> > Patch looks ok as far as I can see (just the parenthesis-addition would<br>
> > be nicer as separate commit), do you have commit access to kde's svn?<br>
> ><br>
> > Andreas<br>
><br>
</div><div class="im">> I'll generate a new patch, but am currently out of time: two exams next week<br>
> :-(<br>
<br>
</div>Don't worry, I'll just commit it as is, but keep that in mind for the next<br>
patch :)<br>
<div class="im"><br>
> By the way: You sent Your mail directly to me and CCed the list, is that<br>
> common practice (means: am I doing sth wrong?) ?<br>
<br>
</div>No, I wasn't sure wether you're subscribed to the list, hence cc'ed you.<br>
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
You will receive a legacy which will place you above want.<br>
</font><div><div></div><div class="h5"><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>
</div></div></blockquote></div><br>You can use <a href="http://reviewboard.kde.org">reviewboard.kde.org</a>, makes it easier to review the patches.<br>