| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
src/core/resources/qframeallocator.cpp
src/plugins/sceneparsers/gltf/gltfparser.cpp
src/render/frontend/qrenderaspect.cpp
tests/auto/render/renderqueue/tst_renderqueue.cpp
Change-Id: Icdc2be2c80e7de6135a09e2f370ee004e395f514
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
all .prf files that need the variables it sets actually load it by
themselves.
This reverts commit 3ec11e4d94d57678f4dd1162185beef62e43da12.
Change-Id: Ia662f252b8215f83e090391748947a8579566885
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
... and mark qgltf as Q_FOREACH-free, using QT_NO_FOREACH.
In writeShaders(), also port two indexed loops to range-for.
Saves almost 6KiB in text size on optimized GCC 6.0 Linux
AMD64 builds.
Change-Id: Id281b077159f461c01c23600f6cb301a6aa34b9e
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The first two lines are implicated in a compile error
in a follow-up commit that introduces C++11 ranged for
loops, so take another shot in the dark and remove the
lambda expression embedded in the QStringLiteral macro
by using QLatin1String instead, which, this being in a
QStringBuilder expression, is even a desirable change
in itself.
Change-Id: I1ccd8d2e3c23cefdd50e7bc71f3f95dbeac78b72
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
All structs are Q_MOVABLE_TYPE, except Node, whose
address is used as its identity, so by definition it
is not trivially relocatable. Therefore explicitly
marked as Q_COMPLEX_TYPE.
Change-Id: I338491a57cc0f70b3971dec34b7dd9d913a92462
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
It's even a true positive. Braces were missing.
Change-Id: Ie807c46659ad4085751bc78b86408ce1001e1daf
Reviewed-by: Frank Meerkoetter <frank.meerkoetter@basyskom.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
| |
| |
| |
| |
| |
| | |
Task-number: QTBUG-52736
Change-Id: I58f9cbcdf018e7b672d33dd865067485412b79fe
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
examples/qt3d/examples-common/qorbitcontrol.h
src/core/qnodecreatedchange.cpp
src/core/qnodecreatedchange.h
src/input/frontend/qaxisactionhandler.h
src/input/frontend/qinputaspect.h
src/input/frontend/qkeyboardhandler.h
src/input/frontend/qmousedevice.h
src/input/frontend/qmousehandler.h
src/input/frontend/qphysicaldevicecreatedchange.h
src/input/input.pro
src/logic/qframeaction.h
src/plugins/sceneparsers/assimp/assimp.pro
src/quick3d/imports/input/importsinput.pro
src/quick3d/imports/render/importsrender.pro
src/render/backend/trianglesextractor.cpp
src/render/framegraph/qclearbuffer.h
src/render/framegraph/qlighting.h
src/render/framegraph/qstateset.h
src/render/frontend/qrenderattachment.h
src/render/geometry/qabstractattribute.h
src/render/geometry/qabstractbuffer.h
src/render/geometry/qattribute.h
src/render/geometry/qboundingvolumespecifier.h
src/render/geometry/qbuffer.h
src/render/materialsystem/qfilterkey.h
src/render/materialsystem/qparameter.h
src/render/materialsystem/qparametermapping.h
src/render/renderstates/qblendstate.h
src/render/renderstates/qdepthmask.h
src/render/renderstates/qpointsize.cpp
src/render/renderstates/qrenderstatecreatedchange.cpp
src/render/renderstates/qstencilop.h
src/render/renderstates/qstencilopseparate.h
src/render/renderstates/qstenciltestseparate.h
src/render/texture/qabstracttexture.h
src/render/texture/qabstracttextureimage.h
src/render/texture/qtextureproviders.h
Change-Id: I894d7781042cabdaa0cac690c198b57a41127bd4
|
| |
| |
| |
| |
| |
| |
| | |
qgltf.cpp:1990:38: error: template argument for ‘template<class Key, class T> class QHash’ uses local type ‘GltfExporter::exportTechniques(QJsonObject&, const QString&)::ProgramNames’
Change-Id: Ic747cc2ab45e4dc6bb70ffff143844dc7120e571
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
src/doc/src/qt3dcollision-module.qdoc
src/render/frontend/qcamera.cpp
src/render/frontend/qcameralens.cpp
Change-Id: I8a0ae250a6b8e065a1ef3c817968ebb7ef23b20d
|
| |
| |
| |
| |
| |
| |
| | |
constBegin() would belong to a different container than constEnd()
Change-Id: I5094d6a6ee9c2167bb69274556f568538c77cf6f
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Updated license headers to use new GPL-EXCEPT header instead of LGPL3 one
(in those files which will be under GPL 3 with exceptions)
Change-Id: I930c6234bfe720d38c596bbc3d1f39be75a7328b
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>
|
|/
|
|
|
| |
Change-Id: I4bee133d1babf86612e584f938938d59de7f422b
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
|
|
|
|
|
|
|
| |
the bootstrap library is supposed to work out of the box now.
Change-Id: Ida1d181b55a37dc9f1b049bc2586aca5a9d9dc9c
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
follow the foo.pri & foo_dependency.pri pattern found in qtbase to
de-duplicate the code (and thus implicitly make it consistent) and
synchronize the conditionals in the configure test.
also document why we still can't enable the assimp sceneparser in all
configurations.
Change-Id: I3bdc30e077b6c9c7027a9311195c08a6c5f1fcf4
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I978127436e197c714f7ad43c86c1c7896b94d9cd
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ie6de66a4150a1ebc8a4a6b555618282b7e756658
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
| |
Due to the KHR_materials_common block Qt 3D is able to load the
generated scenes without the presence of shaders, programs and
techniques blocks. Add an option to skip outputting these,
in order to reduce the size and speed of parsing.
Change-Id: I212e4e66197489c107ee47dcbc33d6f5b6b40156
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
| |
Once again, the problem surfaces: when not specifying an output
directory, image files are not added to the resource list. Change
it to go through the external texture list always.
Change-Id: Ic3da5d7f8b1da958c6453f18650f2e50c70a0d1f
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
| |
The spec uses FLOAT_VEC4 for colors. This means that while in the
custom material case we can use vec3 since we provide the type
information, the common materials path needs to promote all colors
to RGBA.
Change-Id: I1ac403807d0d6f0e9dfe14ea704c7f4a325495fa
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
| |
Let's avoid future confusion by using a custom extension for the
main JSON document. In case assimp gets upgraded to a future version
supporting glTF files for example, confusion can arise due to the
conflicting file extensions in the scene parser plugins.
Change-Id: I370cc45cc399cae4f425d86574ccea9cd6c59d4f
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Unless disabled by a command-line switch.
The spec is insufficient to support normal maps, the "normalmap"
value is our own extension to the extension for now.
Also, the common technique name is always PHONG. The correct material
can be chosen based on the presence and type of the value list entries.
Task-number: QTBUG-49384
Change-Id: I4f2e5848ad8019c44e9392416501655ab1163d7c
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
| |
Change-Id: I050e45a8b2e884ec9ee805fa9ddcbafe705802c3
Reviewed-by: Mike Krus <mike.krus@kdab.com>
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
| |
when we're not cross-building, even a host build may use the system
zlib.
this also makes the force_bootstrap workaround in qgltf superfluous.
Change-Id: Ic0a8b872ac602b1f268a1c6f8dcd5b400d641ff5
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
|
|
|
|
|
|
|
| |
Seems to work on Windows, Linux and OS X.
Change-Id: Ie0a00c4abf12c3dc7451019fc4d53a15094d558a
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Without this commit, the link line for qgltf tries to search in /usr/lib
before the qtbase/lib dir of the Qt currently being built. In doing so
it found the system installed QtCore which is not what we want. In my
case it failed because of missing symbols as I'm using a namespaced
build but in other cases it may lead to more subtle errors.
Using PKGCONFIG_PRIVATE puts the /usr/lib search path from pkgconfig at
the end rather than the beginning of the search path list.
Change-Id: I0f715564c3255100a42d6e9c12b1d10168cd05bb
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
|
|
|
|
|
| |
Change-Id: I137b5b67e89930a055360e3017ce9f0c0ceb6d6b
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
|
|
| |
Task-number: QTBUG-49343
Change-Id: Ice78fc0a689251ddd27f1b4cfe9b5ba2a4db0f0f
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@theqtcompany.com>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that the qmake rule uses "." as the default output directory for the
generated files, external assets like textures are often not copied as they
are in the same place already. However, we still need to list everything in
the generated .qrc file.
This should fix not having textures in many cases with the simplified prf.
Change-Id: I2b593adf9ac9dddf32bf1e411bd9443025727f8f
Reviewed-by: Mika Salmela <mika.salmela@theqtcompany.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
It was wrong of us to assume that a QT_CONFIG containing system-zlib
meant that we could use zlib for a host_build. There are cases where
the host does not have zlib, but the target does (cross compilation). It
also breaks the rules for host_build tools. Fortunately the bootstrap
library does provide the zlib symbols, so we just need to make sure we
include that.
Change-Id: Ibe25a22676c47909d95a83df46e311bd486fc9f7
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
| |
This is a host tool, so we can't use the system
library from the target system when cross-compliing.
Otherwise build with the system library when it's
available.
Change-Id: Id78ba938bda578dc891e2931ced5cf710b0ab7ce
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I1d7241ba8177c664b5d4df4ec50afd2a7c830ff2
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
|
|
|
|
|
|
|
|
|
|
| |
It isn't yet possible to install qmake features from modules, so it
has been moved into qtbase. This make it actually possible to use the
qgltf feature on any platform now (not just places where you ran 'make
install').
Change-Id: I562ff98f71562c8f023437905be3e8725953fc7e
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
|
|
|
|
|
|
|
|
| |
Call load(qt_tool) before including the assimp.pri so it can find
the zlib headers in the install dir.
Change-Id: Icb463a83d08d93e93490b68320ab85aa43769d94
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: I450e55536e8cc234bcec827fea441a8194b19345
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
|
|
|
|
|
|
|
|
|
|
| |
qgltf.cpp(2125): warning C4267: '=': conversion from 'size_t' to 'uint', possible loss of data
qgltf.cpp(2135): warning C4267: '=': conversion from 'size_t' to 'uint', possible loss of data
qgltf.cpp(2152): warning C4267: '=': conversion from 'size_t' to 'uint', possible loss of data
qgltf.cpp(2170): warning C4267: '=': conversion from 'size_t' to 'uint', possible loss of data
qgltf.cpp(2201): warning C4267: '=': conversion from 'size_t' to 'uint', possible loss of data
Change-Id: I7d0f9054bd2736698ec6d13f6eff20180c3262cb
Reviewed-by: Laszlo Agocs <laszlo.agocs@theqtcompany.com>
|
|
|
|
|
|
|
|
| |
This will cause the loader not to enable automatic mipmap generation
which is good because that would fail with GLES.
Change-Id: Ie3d0ea473e28bddc9144aa629019d4331b505829
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
| |
Change-Id: Ic4615231f93ed53b67ed6091497cfd90e8a20cbc
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
|
| |
When the installed bin directory is not in PATH, the tool was
not found. The correct approach is to use qtPrepareTool().
Change-Id: I5f51a950975bc9372eac9e401bd3b89205a9ccd0
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
And remove force_bootstrap that breaks non-cross-compiled
builds.
In true cross-compiled builds cross_compile (that is set
during configure) will trigger force_bootstrap so we don't
have to set that manually for tools outside qtbase.
Change-Id: Ie3bc6fc105ea7686a3098c6afd631561755aa01b
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>
|
|
|
|
|
|
|
|
| |
If needed we can fetch stuff back from master branch.
Change-Id: Ic35f989de4a59d2c0a18ca6b18f0c0aca64d5046
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
|
|
|
|
|
| |
Change-Id: Id7240632c7a4d6e59666488858b23a3fead389cd
Reviewed-by: Giuseppe D'Angelo <giuseppe.dangelo@kdab.com>
|
|
|
|
|
|
|
|
| |
Change-Id: I53de6de78a982968c31a9c8d17628d1cddb63a14
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>
Reviewed-by: Tasuku Suzuki <stasuku@gmail.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
|
|
|
|
|
| |
Change-Id: I4882e2d9fa270e39958a84c698681b938510c05f
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
|
|
|
|
|
|
|
|
| |
Change copyrights and license headers from Nokia to Digia
Change-Id: Ib09067b91869c2840710bffee104bc63c76b0bf7
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>
|
|
|
|
|
|
|
|
|
|
| |
This commit changes the API of all classes in this module to use float
rather than qreal. This makes the API consistent across platforms and
improves floating point performance.
Change-Id: I95ae65cb31428748e066d5efca059ce24087fafa
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>
Reviewed-by: Sarah Jane Smith <sez@storybridge.org>
|
|
|
|
|
|
|
| |
the recent enabling of EGL for desktop triggered this obsolete path.
Change-Id: Ib290c35b48db3b270047b54fe9dc28bbe3c39ea3
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
|
|
|
|
|
|
|
| |
there is nothing to install/package in subdirs projects
Change-Id: I5647914cdfc8231def4386dc9c3946fee4049743
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
|
|
|
|
|
| |
Change-Id: Id6277973e42597d446dfe8f4ce935611a7351c29
Reviewed-by: Joerg Bornemann <joerg.bornemann@nokia.com>
|
|
|
|
|
| |
Change-Id: I62f3d32f326fbab483362ec2a2aad146314570fa
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
|