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/collada/SkinInfo.cpp | |
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/collada/SkinInfo.cpp')
-rw-r--r-- | source/blender/io/collada/SkinInfo.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/source/blender/io/collada/SkinInfo.cpp b/source/blender/io/collada/SkinInfo.cpp index 8f6f1e467d9..12dee388a58 100644 --- a/source/blender/io/collada/SkinInfo.cpp +++ b/source/blender/io/collada/SkinInfo.cpp @@ -55,10 +55,7 @@ template<class T> static const char *bc_get_joint_name(T *node) /* This is used to store data passed in write_controller_data. * Arrays from COLLADAFW::SkinControllerData lose ownership, so do this class members * so that arrays don't get freed until we free them explicitly. */ -SkinInfo::SkinInfo() -{ - /* pass */ -} +SkinInfo::SkinInfo() = default; SkinInfo::SkinInfo(const SkinInfo &skin) : weights(skin.weights), |