<div class="gmail_quote">On Sun, Aug 9, 2009 at 6:36 PM, Stefan Valouch <span dir="ltr"><<a href="mailto:stefan@valouch.de">stefan@valouch.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><div></div><div class="h5">On Sunday 09 August 2009 18:18:05 Aleix Pol wrote:<br>
> On Sun, Aug 9, 2009 at 6:03 PM, Stefan Valouch <<a href="mailto:stefan@valouch.de">stefan@valouch.de</a>> wrote:<br>
> > Hi,<br>
> > the attached patch silents a bunch of warnings from g++ (4.4.1).<br>
> > Mostly, it puts Q_UNUSED() around parameters and add some brackets where<br>
> > g++<br>
> > suggested them. I tried to stay conform to the formatting styles around<br>
> > the changed positions.<br>
> > On positions where the parameters were just used in Q_ASSERTs, I did not<br>
> > add<br>
> > Q_UNUSED, cause I did not want to break something.<br>
> > Additionally, a comment is added at<br>
> > kdevplatform/language/editor/modificationrevision.cpp about the include<br>
> > of a<br>
> > (now) deprecated header.<br>
> ><br>
> > Compiles successfully and KDevelop is happy with it.<br>
> ><br>
> > Stefan<br>
> ><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>
> > It would be much better if you posted that on the reviewboard.<br>
><br>
> I just browsed over the patch nad doesn't look bad to me.<br>
><br>
> Do you have a svn account?<br>
><br>
<br>
</div></div>No, I'm not a KDE developer :-)<br>
<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>Well, the only thing that stops you from being a kde developer is that account. If you want to work on any code in kdevelop, it would be good you got one.<br>