Fix style issues in Java code

Try to make it more consistent both within itself and with
Qt code in general. Add some air.

Change-Id: I32bd3f17e44d912293e990100a53adb7bbd80dc1
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
Add FIXME comment to configure

To make it clear that this is a temporary work-around for a
missing fix in Qt Creator.

Change-Id: Ic2239ca63760fa4c7127967872bb1401f94795f6
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
Remove debug output from Android code

If put back, they should be put inside appropriate #ifdefs,
but for now it seems reasonable to remove them, since they
seem to be very specifically printing information needed
for development.

Change-Id: Ifd541845a694749ed1d59c23d71096be25e50045
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
Revert whitespace changes

The Android branch accidentally changes whitespace in
a few places for no good reason, which just adds noise
to the patch. I've reverted these.

Change-Id: Idc9e7e1a493536a42bca190be1696e8c8312e9f4
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
Remove unneeded option from compile.test script

This was added at some point and does not seem to be needed
anymore.

Change-Id: Id78266d39e154fc13a4a51790f1af8bfaf475a3f
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>
Android style style fixes

Change-Id: I76a4f2547bc646d6d608f1f750619fb2a662ce3e
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@digia.com>
Revert one more whitespace change

The Android branch accidentally changes whitespace in
a few places for no good reason, which just adds noise
to the patch. Here is one more revert.

Change-Id: I8acf2f96c01ba9c031b20edd22280f7adfef5943
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@digia.com>
Revert Android condition from dead code

The xml patterns and webkit modules have been separated out
into separate modules, so these arguments are now unused.
That means there's no reason to make any changes here for
Android.

Change-Id: I3cd8a86cbc3d40c5117c21ad0a6365b13084fd0a
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Android: code style fixes

Change-Id: I7681b99189b65d4d9bebce086ce6494d07538a4e
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@digia.com>
Move CONFIG+=android_app from configure into compile.test

The original change copied the pattern of the Mac SDK hack for
getting compile.test to work, and now that has gone away so
we should remove our hack as well. Since this is something
specific to the compile.test script and since the android_app
CONFIG will not have any effect unless android.prf is loaded,
we can just hardcode it in the script.

Change-Id: I29c8910bb36b2a1e7a26d384d09cbcaaff7207ad
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Fix configure tests for Android toolchains

The Android NDK host existence test in configure
would circumvent everything. It would show and exit
with an error even when you specified the help option,
and it would show the same error even if the problem
was that the default toolchain could not be found.

We need to move the test next to the others so that
it's tested regardless of how the architecture was specified
and so that it's not tested when help is specified, and we
need to check for the existence of the toolchain before
testing for the host architecture.

Change-Id: I5fa763ed69ea58ea025a8d98593e527ce0dd3082
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Paul Olav Tvete <paul.tvete@digia.com>