diff options
author | Jacques Lucke <jacques@blender.org> | 2021-04-08 12:07:12 +0300 |
---|---|---|
committer | Jacques Lucke <jacques@blender.org> | 2021-04-08 12:07:27 +0300 |
commit | 19dfb6ea1f6745c0dbc2ce21839c30184b553878 (patch) | |
tree | 4deac8e505cf2f422ab20d132395d006dddb29c3 /source/blender/io/alembic/exporter | |
parent | 0ea66039dd5374e23e9e60e8e3192f919d4f1abd (diff) |
Cleanup: enable modernize-use-equals-default check
This removes a lot of unnecessary code that is generated by
the compiler automatically.
In very few cases, a defaulted destructor in a .cc file is
still necessary, because of forward declarations in the header.
I removed some defaulted virtual destructors, because they are not
necessary, when the parent class has a virtual destructor already.
Defaulted constructors are only necessary when there is another
constructor, but the class should still be default constructible.
Differential Revision: https://developer.blender.org/D10911
Diffstat (limited to 'source/blender/io/alembic/exporter')
8 files changed, 1 insertions, 20 deletions
diff --git a/source/blender/io/alembic/exporter/abc_custom_props.cc b/source/blender/io/alembic/exporter/abc_custom_props.cc index 382afdc294d..f5593e7ee30 100644 --- a/source/blender/io/alembic/exporter/abc_custom_props.cc +++ b/source/blender/io/alembic/exporter/abc_custom_props.cc @@ -51,10 +51,6 @@ CustomPropertiesExporter::CustomPropertiesExporter(ABCAbstractWriter *owner) : o { } -CustomPropertiesExporter::~CustomPropertiesExporter() -{ -} - void CustomPropertiesExporter::write_all(const IDProperty *group) { if (group == nullptr) { diff --git a/source/blender/io/alembic/exporter/abc_custom_props.h b/source/blender/io/alembic/exporter/abc_custom_props.h index d3f9b2fc43c..fcd47382d96 100644 --- a/source/blender/io/alembic/exporter/abc_custom_props.h +++ b/source/blender/io/alembic/exporter/abc_custom_props.h @@ -61,7 +61,7 @@ class CustomPropertiesExporter { public: CustomPropertiesExporter(ABCAbstractWriter *owner); - virtual ~CustomPropertiesExporter(); + virtual ~CustomPropertiesExporter() = default; void write_all(const IDProperty *group); diff --git a/source/blender/io/alembic/exporter/abc_writer_abstract.cc b/source/blender/io/alembic/exporter/abc_writer_abstract.cc index e99048cc0ee..27b5c2fa2a4 100644 --- a/source/blender/io/alembic/exporter/abc_writer_abstract.cc +++ b/source/blender/io/alembic/exporter/abc_writer_abstract.cc @@ -45,10 +45,6 @@ ABCAbstractWriter::ABCAbstractWriter(const ABCWriterConstructorArgs &args) { } -ABCAbstractWriter::~ABCAbstractWriter() -{ -} - bool ABCAbstractWriter::is_supported(const HierarchyContext * /*context*/) const { return true; diff --git a/source/blender/io/alembic/exporter/abc_writer_abstract.h b/source/blender/io/alembic/exporter/abc_writer_abstract.h index 4997e498199..d3500394555 100644 --- a/source/blender/io/alembic/exporter/abc_writer_abstract.h +++ b/source/blender/io/alembic/exporter/abc_writer_abstract.h @@ -50,7 +50,6 @@ class ABCAbstractWriter : public AbstractHierarchyWriter { public: explicit ABCAbstractWriter(const ABCWriterConstructorArgs &args); - virtual ~ABCAbstractWriter(); virtual void write(HierarchyContext &context) override; diff --git a/source/blender/io/alembic/exporter/abc_writer_instance.cc b/source/blender/io/alembic/exporter/abc_writer_instance.cc index 7f3b044cb8b..1737e8c091e 100644 --- a/source/blender/io/alembic/exporter/abc_writer_instance.cc +++ b/source/blender/io/alembic/exporter/abc_writer_instance.cc @@ -35,10 +35,6 @@ ABCInstanceWriter::ABCInstanceWriter(const ABCWriterConstructorArgs &args) { } -ABCInstanceWriter::~ABCInstanceWriter() -{ -} - void ABCInstanceWriter::create_alembic_objects(const HierarchyContext *context) { OObject original = args_.hierarchy_iterator->get_alembic_object(context->original_export_path); diff --git a/source/blender/io/alembic/exporter/abc_writer_instance.h b/source/blender/io/alembic/exporter/abc_writer_instance.h index f7d6450055a..1e4285d5e02 100644 --- a/source/blender/io/alembic/exporter/abc_writer_instance.h +++ b/source/blender/io/alembic/exporter/abc_writer_instance.h @@ -31,7 +31,6 @@ namespace blender::io::alembic { class ABCInstanceWriter : public ABCAbstractWriter { public: explicit ABCInstanceWriter(const ABCWriterConstructorArgs &args); - virtual ~ABCInstanceWriter(); virtual void create_alembic_objects(const HierarchyContext *context) override; virtual Alembic::Abc::OObject get_alembic_object() const override; diff --git a/source/blender/io/alembic/exporter/abc_writer_mesh.cc b/source/blender/io/alembic/exporter/abc_writer_mesh.cc index fbc662113cc..c00e57c8edc 100644 --- a/source/blender/io/alembic/exporter/abc_writer_mesh.cc +++ b/source/blender/io/alembic/exporter/abc_writer_mesh.cc @@ -113,10 +113,6 @@ void ABCGenericMeshWriter::create_alembic_objects(const HierarchyContext *contex liquid_sim_modifier_ = get_liquid_sim_modifier(scene_eval, context->object); } -ABCGenericMeshWriter::~ABCGenericMeshWriter() -{ -} - Alembic::Abc::OObject ABCGenericMeshWriter::get_alembic_object() const { if (is_subd_) { diff --git a/source/blender/io/alembic/exporter/abc_writer_mesh.h b/source/blender/io/alembic/exporter/abc_writer_mesh.h index ed4fb4e4514..0e1792b9dab 100644 --- a/source/blender/io/alembic/exporter/abc_writer_mesh.h +++ b/source/blender/io/alembic/exporter/abc_writer_mesh.h @@ -51,7 +51,6 @@ class ABCGenericMeshWriter : public ABCAbstractWriter { public: explicit ABCGenericMeshWriter(const ABCWriterConstructorArgs &args); - virtual ~ABCGenericMeshWriter(); virtual void create_alembic_objects(const HierarchyContext *context) override; virtual Alembic::Abc::OObject get_alembic_object() const override; |