Many thanks Daniel.<div><br></div><div>I applied and pushed these commits to 2.3-stagging.</div><div><br></div><div>Best regards,</div><div><br></div><div>Ray.</div><div><br><div class="gmail_quote">On Thu, Sep 1, 2011 at 7:24 PM, Daniel Teske <span dir="ltr"><<a href="mailto:daniel.teske@nokia.com">daniel.teske@nokia.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Thursday 01 Sep 2011 15:43:05 Teske Daniel (Nokia-MP-Qt/Berlin) wrote:<br>
> > I tried to apply your patches 2.3-stagin but it doesn't compile anymore,<br>
> > should I create another branch and merge master branch of upstream<br>
> > qt-creator ? This mean to create another merge request !<br>
><br>
> The patches were on top of a master merge into 2.3-staging.<br>
> So 0002 and 0003 should apply cleanly, whereas you 0001 is only needed if<br>
> you want to merge master into 2.3-staging. That depends on your plans for<br>
> the next sdk release. I'd guess you would probably want to use 2.3 for<br>
> now, which means, you should save 0001 for later.<br>
<br>
</div>I did recreate those patches against 2.3-staging and added a few more:<br>
0001 and 0002 are just the rebased patches.<br>
<br>
0003:<br>
I did a first pass over the android plugin code and fixed all coding style<br>
errors I found. Please check this patch carefully, I'm pretty sure that with<br>
the amount of small changes, I might have made a conversion error somewhere.<br>
The major ones were:<br>
- put a space after "if", "while", "for" and others.<br>
- put spaces around operators like "+", "-", "+="<br>
- put a space after "," and ";" in the for loop<br>
- use isEmpty() instead if size() or lenght() (for no particular reason<br>
actually)<br>
- "Type &variable" is our prefered way<br>
- braces (except for classes and functions) don't have a line for themselves<br>
- the branches in if/else, either both or none should have braces<br>
Various small things<br>
I also probably missed quite a few cases, but that's a start. :)<br>
<br>
0004:<br>
For some reason, without that patch the branch doesn't compile for me<br>
<br>
0005:<br>
QLatin1String is essentially just a const char *, thus passing a reference to<br>
it, is the same as passing a reference to a pointer, that's not faster then<br>
passing the pointer directly.<br>
<br>
0006:<br>
I think the QProcess is leaked if the process could not be started for any<br>
reason, since in that case no finished signal is emitted.<br>
<br>
0007:<br>
The using isEnabled() function was to work around that virtual functions get<br>
hidden if they are overloaded in a derived class with a different parameter<br>
list. I removed that problem before 2.3, there's now no overloading anymore,<br>
thus the using line is simply no longer needed<br>
<br>
0008:<br>
The cast is not needed if we pass on the right type<br>
<br>
0009:<br>
The class doesn't exist anymore in that namespace, and indeed it wasn't used<br>
anymore<br>
<br>
0010:<br>
The code looked wrong, though I'm not sure whether that fix is the correct one.<br>
I haven't actually tested it.<br>
<br>
0011:<br>
The top level <a href="http://qtcreator.pro" target="_blank">qtcreator.pro</a> checks whether qt is at least 4.7.4, thus the code<br>
can be savely removed<br>
<br>
0012:<br>
We don't use abbreviations in our names, thus "Dont" isn't allowed, also<br>
NoMatch seems to be a better fit. The other changes should be pretty clear.<br>
<font color="#888888"><br>
daniel<br>
</font><br>_______________________________________________<br>
Necessitas-devel mailing list<br>
<a href="mailto:Necessitas-devel@kde.org">Necessitas-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/necessitas-devel" target="_blank">https://mail.kde.org/mailman/listinfo/necessitas-devel</a><br>
<br></blockquote></div><br></div>