Change-Id: Id5ab54370df40a5a285b3ffb6742ef1ec186f144
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Calling cleanupNodes() in the destructor of the scene manager will
cause a crash, as some of the nodes will try to release their
GL resources when theres no valid context to be used for that.
This change is a simple workaround, a more appropriate
change would be to get the destruction correct and clean-up properly,
but that's going to be a bit more involved, so for now it's better
to accept bad clean-up at exit then a crash.

Fixes: QTBUG-82427
Change-Id: I7a2ad8850c27f5544547f6bf207755dd7a1b9213
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Task-number: QTBUG-82427
Change-Id: I564e09d5b0f1515236f98273f57e28fc5b4dc5d1
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

The Screen blendmode is defined as:
1-(1-SRC)*(1-DST)
Which is the same as:
SRC + DST(1-SRC) or in glBlendFunc terms:
GL_ONE, GL_ONE_MINUS_SRC_COLOR

Output should now be the expected result, but should break the lancelot
test.

Fixes: QTBUG-77786
Change-Id: I01ef0b1e08389394f1a6dc7019587a3473f22f5f
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

This solves a compilation error when it is built against older version of Qt.

Change-Id: I7e37bdafbe81e34310400d6e721076e4f96299eb
Reviewed-by: Antti Määttä <antti.maatta@qt.io>

Andy Nichols authored 5 years ago and Andy Nichols committed 5 years ago

It doesn't make sense to build any tooling or plugins for the importing
of assets on non-development platforms. This tooling is not intended to
be used at runtime, so don't even bother building it.

Fixes: QTBUG-80572
Change-Id: I442a82df6d299fdd13d9c02faeaeace3f2553e7c
Reviewed-by: Inho Lee <inho.lee@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

modifies light stuffs including AreaLight

Change-Id: I96c70678f7f8bc9db2174ff2e98243993adc276e
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Antti Määttä <antti.maatta@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Made the class, subjectively, easier to use and understand.
We don't use this class a lot, but we have some good candidates where
we could use it in the future, but that will be considered later.
Added tests to make sure we have good testing coverage.

Change-Id: If31a9f7851c4556d85c64bf2754fb08d53bc7f38
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Task-number: QTBUG-82805
Change-Id: I8610f20ddaffd0f6fdffaffafa747552a9051a2f
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

...independently of screen scale factor.

Change-Id: Ia026f5df1017b93551e9cb4ed1b12a32bf757215
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Paul Tvete authored 5 years ago and Christian Strømme committed 5 years ago

Not all drivers/devices support all compressed texture formats, so
restore the old version of this example that uses jpgs. Copy the
current version to the manual tests directory.

Change-Id: I2f3f013f915f16aa6903ffef195c942b1ec23306
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
(cherry picked from commit 9b402cdf)

This module is not functional without OpenGL support in 5.x, so just
tell builds to skip this module if Qt was configure without OpenGL
support.

Fixes: QTBUG-81052
Change-Id: I65aa73a94d9beaf0a58c24cb032e1303a2e0c4c8
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Design Studio can't deal with quaternion rotations, so import rotations
as Euler.

Task-number: QDS-1813
Change-Id: Ibc10012a42f9a7c891ea0b95a6ac11d02767afb2
Reviewed-by: Mahmoud Badri <mahmoud.badri@qt.io>
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>

Task-number: QTBUG-82905
Change-Id: I04855c7aeda2e0c31ac34e10e68dfef1db8b9a47
Reviewed-by: Jere Tuliniemi <jere.tuliniemi@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

After these chages lancelot MultipleItems test renders correctly.
Also adding new test to confirm the rendering ordering with multiple
nodes and mixing 2D items with 3D models.

Task-number: QTBUG-82430
Change-Id: I5aa17b3ff804e535a315acda7c9a97caccfafb8d
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Change-Id: Ifcd7b88815609a9346a1f2d2b7504bef6d2aefe1
Reviewed-by: Mahmoud Badri <mahmoud.badri@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

Removing also some other leftovers from QSSGRendererImpl

Task-number: QTBUG-82740
Change-Id: If6b5e8ff9be233557843c70d27d80df6099de7dc
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Antti Määttä <antti.maatta@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Instead of discarding full-alpha pixels, use blending.

Task-number: QTBUG-82456
Change-Id: I5dd791ddeab615c6879f4fd1423cdd4f9b3958f5
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Inho Lee <inho.lee@qt.io>
Reviewed-by: Antti Määttä <antti.maatta@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Fix Item2D visibility and opacity to work as expected. Now passes all
lancelot quick/Visibility.qml test cases. Remove some unused code.

Task-number: QTBUG-82429
Task-number: QTBUG-82679
Change-Id: Id9582ef3f37ffdae1658cf48b6eb0516e372a795
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Laszlo Agocs authored 5 years ago and Christian Strømme committed 5 years ago

Not tested in any form, yet it is in the public API. Have one test case at
least.

Change-Id: Ic016a6ce62fc6a5cbdf79e713393ee38b4bf52d2
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Exercise having a custom material based object in combination with lights,
shadow mapping, and SSAO.

The results are questionable when it comes to lighting, see QTBUG-82587,
but as long as we are identical with and without RHI, that's already
something.

Change-Id: I6e484ad559207e8ece26e1646a0be9e09c987a08
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Task-number: QTBUG-82439
Change-Id: I62e730a994546f5b66732c98ca08bb46557f03c4
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Change-Id: I2562aeb0efed581fb7d5f57df233ebc030a1ab3f
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

- Renamed property cullingMode to cullMode
- Renamed enums from XCullface to XCullFace
- Renamed DisableCulling to NoCulling
- Removed FrontAndBackfaceCulling from the QML API

Change-Id: Ieab42ea6609a6c3f19bc6356403d487aacfdf23d
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>

It is not quite useful to demonstrate Qt Graphical Effects, especially
when the name (blend modes) misleads people thinking about blending within
the 3D content.

Graphical Effects may also not be a supported module in 6.0 so relying on
it should be avoided.

Task-number: QTBUG-82283
Change-Id: I5f1af731410add6cedd52f8fe952ebef2991e00a
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>

Change-Id: Ib131ed865aab6d8cf100a35c8d511540f2df6ed8
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

Check if we've received a scene manager before setting it on the
materials. If no scene manager has been set defer until one is set.

Fixes: QTBUG-82688
Change-Id: I642a27ac21f0e1f40d6a32d8286b6991ffbba459
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Task-number: QTBUG-82439
Change-Id: Iccb5a6b1473a077021c671d8e81ac75f522352e5
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Remove layer from textures before deleting it.
Also don't delete m_layer directly to avoid QOpenGLContext assert.

Task-number: QTBUG-82622
Task-number: QTBUG-82613
Change-Id: I7ea374d7195d0399bca4c8047b16c3361ad19ddf
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

TextureProvider items handled similarly as non-textureProviders.

Task-number: QTBUG-82622
Change-Id: I4935930d5ea55b0f37ca61d66c33183a1178fd28
Reviewed-by: Inho Lee <inho.lee@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Task-number: QDS-1694
Change-Id: I0a441fb693331e2673b6d4a597ac9fef5ef0dff9
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>

Make sure the material has a scene manager set. In most cases
it's sufficient to set the parent item of the material, if one is not
set already. If the material has no parent item set, check if it has
an immediate hierarchical parent that can be used, this is normally
the case for inline materials, materials declared at some random
place in the scene or properties. There are other cases where the
immediate parent is of a non-suitable type and we should handle
that as well, by manually updating the material with the model's
scene manager through the ref-/derefSceneManager() functions.

Fixes: QTBUG-82276
Change-Id: Id0d758235376520374c288712dd220e901a76aeb
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

The type() function cannot be a virtual function, we use this function
several places for run-time type information, and most critically in
the destruction of the objects, so it needs to be stable!
In the base class the function was pure virtual, which meant we would
in cases call the pure virtual function and suffer the consequences.

Change-Id: I530744cf9ff818309b4a8b5141ff6567c25e6914
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>

... and make it protected

Change-Id: Ib62c9eb0746038fc14dbdb46efe66339d2a9e1de
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

There's no use-case for this as far as I can tell, and handing out a
raw point from a QSharedPointer is far from ideal.
The scene manager can be retrieved through the private implementation,
if needed. Also removed the sceneManagerChanged() signal, it's not
used and we already have infrastructure to notify scene manager changes.

Change-Id: Ie0c4f72f60631ff2dd697be22bb3d9251af45f1f
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Inho Lee <inho.lee@qt.io>
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Reviewed-by: Andy Nichols <andy.nichols@qt.io>

Fixes: QTBUG-82576
Change-Id: I2d3c033f3785f0e81cdcd1bfb38a244495f6b560
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Default to not using pipe on Windows, with proper temp path.

Task-number: QTBUG-82439
Change-Id: Ifc30ed5229deb19d877ba2b1306b09238562a2d8
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>

This is not planned to be supported in the foreseeable future, so the conversion
will always be necessary if the input is, for example, a 16 bits per component PNG
file.

If we gain support for RGBA16 texture formats in the graphics stack in the
future (a low priority feature for now), the conversion can be removed.
But until then, there is no point in showering warnings all the time.

Change-Id: Ie6f9b977a6f9ec5d6ae18123eeb7053ce2ad4491
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Fixes: QTBUG-82575
Change-Id: I97ed94b71e969dfcdb87ef276a6aeb9fce2e7227
Reviewed-by: Christian Strømme <christian.stromme@qt.io>

Change-Id: I5ec116592c52d337d3dcb6296d10d2cfd229639c
Reviewed-by: Paul Wicking <paul.wicking@qt.io>