<div class="gmail_quote">On Sun, Aug 9, 2009 at 6:03 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;">
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 g++<br>
suggested them. I tried to stay conform to the formatting styles around the<br>
changed positions.<br>
On positions where the parameters were just used in Q_ASSERTs, I did not 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 of a<br>
(now) deprecated header.<br>
<br>
Compiles successfully and KDevelop is happy with it.<br>
<font color="#888888"><br>
Stefan<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>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>