Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
erase inheritance from QObject
  • Loading branch information
NEDJIMAbelgacem committed Jul 27, 2020
1 parent 64f004b commit fd8c68c
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 17 deletions.
5 changes: 2 additions & 3 deletions src/3d/qgs3dexportobject.h
Expand Up @@ -32,9 +32,8 @@ class QgsAbstractMaterialSettings;
* \ingroup 3d
* \since QGIS 3.16
*/
class Qgs3DExportObject : public QObject
class Qgs3DExportObject
{
Q_OBJECT
public:
//! The type of exported geometry
enum ObjectType
Expand All @@ -52,7 +51,7 @@ class Qgs3DExportObject : public QObject
* \param parent
* The parent QObject (we use this to delete the Qgs3DExportObject instance once the exporter instance is deallocated)
*/
Qgs3DExportObject( const QString &name, QObject *parent = nullptr ) : QObject( parent ), mName( name ) { }
Qgs3DExportObject( const QString &name ) : mName( name ) { }

//! Returns the object name
QString name() const { return mName; }
Expand Down
2 changes: 1 addition & 1 deletion src/3d/qgs3dmapexportsettings.cpp
Expand Up @@ -16,7 +16,7 @@
#include "qgs3dmapexportsettings.h"
#include "qgssettings.h"

Qgs3DMapExportSettings::Qgs3DMapExportSettings( QObject *parent ): QObject( parent )
Qgs3DMapExportSettings::Qgs3DMapExportSettings()
{
QgsSettings settings;
mSceneName = settings.value( QStringLiteral( "UI/last3DSceneExportName" ), QStringLiteral( "Scene" ) ).toString();
Expand Down
5 changes: 2 additions & 3 deletions src/3d/qgs3dmapexportsettings.h
Expand Up @@ -28,12 +28,11 @@
* \ingroup 3d
* \since QGIS 3.16
*/
class _3D_EXPORT Qgs3DMapExportSettings : public QObject
class _3D_EXPORT Qgs3DMapExportSettings
{
Q_OBJECT
public:
//! Constructor
Qgs3DMapExportSettings( QObject *parent = nullptr );
Qgs3DMapExportSettings();

//! destructor (save the export settings before deallocation)
~Qgs3DMapExportSettings();
Expand Down
4 changes: 2 additions & 2 deletions src/3d/qgs3dmapscene.cpp
Expand Up @@ -899,14 +899,14 @@ void Qgs3DMapScene::exportScene( const Qgs3DMapExportSettings &exportSettings )
}
}

if ( mTerrain != nullptr )
if ( mTerrain )
exporter.parseTerrain( mTerrain, "Terrain" );

exporter.save( exportSettings.sceneName(), exportSettings.sceneFolderPath() );

if ( !notParsedLayers.empty() )
{
QString message = "The following layers were not exported: \n";
QString message = tr( "The following layers were not exported:" ) + "\n";
for ( const QString &layerName : notParsedLayers )
message += layerName + "\n";
QgsMessageOutput::showMessage( tr( "3D exporter warning" ), message, QgsMessageOutput::MessageText );
Expand Down
12 changes: 6 additions & 6 deletions src/3d/qgs3dsceneexporter.cpp
Expand Up @@ -428,7 +428,7 @@ void Qgs3DSceneExporter::parseFlatTile( QgsTerrainTileEntity *tileEntity, const
QString objectNamePrefix = layerName;
if ( objectNamePrefix != QString() ) objectNamePrefix += QString();

Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "Flat_tile" ) ), this );
Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "Flat_tile" ) ) );
mObjects.push_back( object );

object->setSmoothEdges( mSmoothEdges );
Expand Down Expand Up @@ -483,7 +483,7 @@ void Qgs3DSceneExporter::parseDemTile( QgsTerrainTileEntity *tileEntity, const Q
QString objectNamePrefix = layerName;
if ( objectNamePrefix != QString() ) objectNamePrefix += QStringLiteral( "_" );

Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( layerName + QStringLiteral( "DEM_tile" ) ), this );
Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( layerName + QStringLiteral( "DEM_tile" ) ) );
mObjects.push_back( object );

object->setSmoothEdges( mSmoothEdges );
Expand Down Expand Up @@ -550,7 +550,7 @@ QVector<Qgs3DExportObject *> Qgs3DSceneExporter::processInstancedPointGeometry(
QVector<float> instancePosition = getAttributeData<float>( instanceDataAttribute, instancePositionBytes );
for ( int i = 0; i < instancePosition.size(); i += 3 )
{
Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "shape_geometry" ) ), this );
Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "shape_geometry" ) ) );
objects.push_back( object );
object->setupPositionCoordinates( positionData, 1.0f, QVector3D( instancePosition[i], instancePosition[i + 1], instancePosition[i + 2] ) );
object->setupFaces( indexData );
Expand Down Expand Up @@ -649,7 +649,7 @@ Qgs3DExportObject *Qgs3DSceneExporter::processGeometryRenderer( Qt3DRender::QGeo
return nullptr;
}

Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "mesh_geometry" ) ), this );
Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "mesh_geometry" ) ) );
object->setupPositionCoordinates( positionData, scale * sceneScale, translation + sceneTranslation );
object->setupFaces( indexData );

Expand Down Expand Up @@ -701,7 +701,7 @@ QVector<Qgs3DExportObject *> Qgs3DSceneExporter::processLines( Qt3DCore::QEntity
QVector<float> positionData = getAttributeData<float>( positionAttribute, vertexBytes );
QVector<uint> indexData = getIndexData( indexAttribute, indexBytes );

Qgs3DExportObject *exportObject = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "line" ) ), this );
Qgs3DExportObject *exportObject = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "line" ) ) );
exportObject->setType( Qgs3DExportObject::LineStrip );
exportObject->setupPositionCoordinates( positionData );
exportObject->setupLine( indexData );
Expand All @@ -727,7 +727,7 @@ Qgs3DExportObject *Qgs3DSceneExporter::processPoints( Qt3DCore::QEntity *entity,
QVector<float> positions = getAttributeData<float>( positionAttribute, positionBytes );
points << positions;
}
Qgs3DExportObject *obj = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "points" ) ), this );
Qgs3DExportObject *obj = new Qgs3DExportObject( getObjectName( objectNamePrefix + QStringLiteral( "points" ) ) );
obj->setType( Qgs3DExportObject::Points );
obj->setupPositionCoordinates( points );
return obj;
Expand Down
11 changes: 9 additions & 2 deletions src/3d/qgs3dsceneexporter.h
Expand Up @@ -25,6 +25,8 @@
#include <QFile>
#include <QVector3D>

#include "qgs3dexportobject.h"

class QgsTessellatedPolygonGeometry;
class QgsTerrainTileEntity;
class QgsTerrainEntity;
Expand All @@ -48,10 +50,15 @@ class QgsMeshEntity;
*/
class Qgs3DSceneExporter : public Qt3DCore::QEntity
{
Q_OBJECT
public:
//! Constructor
Qgs3DSceneExporter( Qt3DCore::QNode *parent = nullptr ) : Qt3DCore::QEntity( parent ) { }
Qgs3DSceneExporter() { }
//! Destructor
~Qgs3DSceneExporter()
{
for ( Qgs3DExportObject *obj : mObjects )
delete obj;
}

/**
* Creates necessary export objects from entity if it represents valid vector layer entity
Expand Down

0 comments on commit fd8c68c

Please sign in to comment.