| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
| |
Using optimize_full gives us -O3 also on icc and clang, and -O3 already
implies -ftree-vectorize, and -ffast-math already implies unsafe-math-
optimizations.
Change-Id: Ieb1a6841da1ea3513ae624bf2468554bc62318c9
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
|
| |
|
|
|
|
|
|
|
|
| |
We don't care to fix them.
clipper.cpp:3665:13: warning: unused variable 'firstLeft' [-Wunused-variable]
sweep.cpp:706:8: warning: reference cannot be bound to dereferenced null pointer in well-defined C++ code; comparison may be assumed to always evaluate to false [-Wtautological-undefined-compare]
Change-Id: I27b55fdf514247549455fffd14b17ba6040ac332
Reviewed-by: Paolo Angelelli <paolo.angelelli@qt.io>
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Add qt_attribution.json files for 3rdparty code in Qt Location, and
list the attributions in the index file. Also document the different
licenses the module is available under as a whole.
The _legal.qdoc files apparently weren't included anywhere, and have
been removed.
Change-Id: I56e8b75ae044954c745b2fde6f7cd4c1e80ec1b3
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
QtLocation makes use of a 3rd party library for polygon tessellation,
poly2tri.
This library is known to be weak in handling complex input, like, for
example, polygons with self intersecting edges.
Sanitizing the input data through clipper solves the problem, and it
has already been implemented in the clip2tri library, which this
patch includes and uses.
Task-number: QTBUG-52076
Change-Id: I071a58e202345bc71da583995f7245361f00e8c4
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
|
| |
|
|
|
|
|
|
|
|
|
|
| |
86549ae5135f2efa9ceb0689b88004ede5269668 imported the poly2tri library
with doubles changed to floats. The poly2tri has expectations on the
precision of floating point operations which are not met with single
precision floating point operations, leading to crashes. Revert back to
using double precision.
Change-Id: Ifd35287461a268fd89cdc6674aa31fef1dd55562
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Reviewed-by: Michal Klocek <michal.klocek@digia.com>
|
| |
|
|
|
|
|
|
| |
Sync'd with upstream change 26242d0aa7b8.
Change-Id: I16e8fd333006b88c464d8e083af181cd2ce726e3
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Reviewed-by: Michal Klocek <michal.klocek@digia.com>
|
| |
|
|
|
|
| |
Task-number: QTBUG-43006
Change-Id: I2143b815d8b597f3a02e8bd18021f6133fa9ea17
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
|
| |
|
|
|
| |
Change-Id: If46057e48040fda7b61b81e735c171daf2cc7d06
Reviewed-by: Jerome Pasion <jerome.pasion@digia.com>
|
| |
|
|
|
|
|
| |
Task-number: QTBUG-31445
Change-Id: Iac7771f06f4a8f784c7330e299fe0bc448e45202
Reviewed-by: Alex <alexander.blasche@digia.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
In some cases the 3rd party project source tree "poly2tri" does contain
an "assert(0)" instead of a return statement. This lets at least the
GNU compiler assume a return value is missing.
Some systems consider this an error. To prevent compile errors on those
this change adds an additional compiler flag to make functions without a valid
return value a warning instead of an error.
Change-Id: I13acda8bfd274e37195d49392f4f93990f57ded2
Reviewed-by: Alex <ablasche@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since the poly2tri library doesn't load qt_module.prf, build_all doesn't
get added to CONFIG automatically. This resulted in only the debug
version of the library getting built when Qt was configured for building
both debug and release.
Fixed by making sure build_all and debug_and_release are added to
CONFIG explicitly in affected libraries if they are present in
QT_CONFIG.
Task-number: QTBUG-27679
Change-Id: I62172fc311383ab11c9a71fb65ccfcd163268b82
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@digia.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
|
| |
|
|
|
|
|
|
| |
including <cmath> declares the standard maths functions in the std
namespace not the global namespace.
Change-Id: I293cba5ed166b4f122bb34bca5c1d508e2dda259
Reviewed-by: Aaron McCarthy <aaron.mccarthy@nokia.com>
|
| |
|
|
|
|
|
|
|
| |
- Unused variables.
- 64bit truncations.
Change-Id: I358a1378e875fea4a0790db3c30e422a0b9eba40
Reviewed-by: abcd <amos.choy@nokia.com>
Reviewed-by: Aaron McCarthy <aaron.mccarthy@nokia.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Apparently math.h in WEC7 does not define M_PI used by qtlocation.
Added the requiresd define the necessary files.
qSqrt returns qreal and it is defined to 'float' on some configurations
and to 'double' in other configurations. In those configirations where
it is defined to float, no suitable overload for division is found.
Added explicit cast for qSqrt return value to double to fix the build.
Change-Id: I734fab66abfd4ce95e02ec83e373f0eacd87bdef
Reviewed-by: Andreas Holzammer <andreas.holzammer@kdab.com>
Reviewed-by: Björn Breitmeyer <bjoern.breitmeyer@kdab.com>
Reviewed-by: Aaron McCarthy <aaron.mccarthy@nokia.com>
|
| |
|
|
|
|
|
|
| |
- Assigned/unused variables.
- Signed/unsigned comparison
Change-Id: I712baa90442f9fc9699965e7c7e1c5f4ca319aab
Reviewed-by: Jean Gressmann <jean.gressmann@nokia.com>
|
|
|
Replacing the qTriangulate call with poly2tri. This is a BSD-licensed
triangulation library that gives better performance for most of our
shapes. See http://pastie.org/3893553 for a timing comparison on
linux-x86 (timing is given for the whole screen transformation step)
The BSD-licensed code is isolated into its own dir (in src/3rdparty)
and compiled as a separate static library to be linked in. The files
have been renamed (.cc => .cpp) and doubles changed to floats, but
are otherwise identical to those found at http://code.google.com/p/poly2tri/
Change-Id: Ib07aadafdc5d4b8ac1024e1eecd70ab39f673197
Reviewed-by: Aaron McCarthy <aaron.mccarthy@nokia.com>
|
