Review from Benoit

src/3d/qgstiledscenechunkloader_p.cpp

@@ -200,9 +200,9 @@ QgsTiledSceneChunkLoaderFactory::QgsTiledSceneChunkLoaderFactory(
   , mIndex( index )
   , mZValueScale( zValueScale )
   , mZValueOffset( zValueOffset )
+  , mLayerCrs( layerCrs )
 {
   mBoundsTransform = QgsCoordinateTransform( tileCrs, context.crs(), context.transformContext() );
-  mLayerCrs = layerCrs;
 }
 
 QgsChunkLoader *QgsTiledSceneChunkLoaderFactory::createChunkLoader( QgsChunkNode *node ) const

src/core/tiledscene/qgsesrii3sdataprovider.cpp

@@ -42,6 +42,11 @@
 
 ///@cond PRIVATE
 
+/**
+ * \ingroup core
+ * Tiled scene index implementation for I3S
+ * \since QGIS 4.0
+ */
 class QgsEsriI3STiledSceneIndex final : public QgsAbstractTiledSceneIndex
 {
   public:
@@ -93,6 +98,13 @@ class QgsEsriI3STiledSceneIndex final : public QgsAbstractTiledSceneIndex
 
 
 
+/**
+ * \ingroup core
+ * Data to be shared between multiple copies of the I3S data provider.
+ * As the class may be used from multiple threads, members should be
+ * accessed only while the read/write lock is locked.
+ * \since QGIS 4.0
+ */
 class QgsEsriI3SDataProviderSharedData
 {
   public:

src/core/tiledscene/qgsesrii3sdataprovider.h

@@ -27,6 +27,11 @@ class QgsEsriI3SDataProviderSharedData;
 
 ///@cond PRIVATE
 
+/**
+ * \ingroup core
+ * Data provider implementation for Esri I3S
+ * \since QGIS 4.0
+ */
 class CORE_EXPORT QgsEsriI3SDataProvider final: public QgsTiledSceneDataProvider
 {
     Q_OBJECT
@@ -66,6 +71,11 @@ class CORE_EXPORT QgsEsriI3SDataProvider final: public QgsTiledSceneDataProvider
 
 };
 
+/**
+ * \ingroup core
+ * Data provider metadata implementation for Esri I3S
+ * \since QGIS 4.0
+ */
 class QgsEsriI3SProviderMetadata : public QgsProviderMetadata
 {
     Q_OBJECT