refactor: Collada reorganize class constructors

- Class constructors without body (only attribute initialisations)
  can safely be kept in the class header files

- Constructor variables should be initialized in the order of their
  definition in the header files

This change is also aimed to remove a couple of
build warnings from the linux builds.

1 files changed, 12 insertions, 4 deletions

diff --git a/source/blender/collada/SceneExporter.h b/source/blender/collada/SceneExporter.h
index f4868f527ab..f6525d5438d 100644
--- a/source/blender/collada/SceneExporter.h
+++ b/source/blender/collada/SceneExporter.h
@@ -94,18 +94,26 @@ extern "C" {
 class SceneExporter: COLLADASW::LibraryVisualScenes, protected TransformWriter, protected InstanceWriter
 {
 public:
-	SceneExporter(BlenderContext &blender_context, COLLADASW::StreamWriter *sw, ArmatureExporter *arm, const ExportSettings *export_settings);
+
+	SceneExporter(BlenderContext &blender_context, COLLADASW::StreamWriter *sw, ArmatureExporter *arm, const ExportSettings *export_settings) :
+		COLLADASW::LibraryVisualScenes(sw),
+		blender_context(blender_context),
+		arm_exporter(arm),
+		export_settings(export_settings)
+	{}
+
 	void exportScene();
 
 private:
-	friend class ArmatureExporter;
 	BlenderContext &blender_context;
+	friend class ArmatureExporter;
+	ArmatureExporter *arm_exporter;
+	const ExportSettings *export_settings;
+
 	void exportHierarchy();
 	void writeNodeList(std::vector<Object *> &child_objects, Object *parent);
 	void writeNodes(Object *ob);
 
-	ArmatureExporter *arm_exporter;
-	const ExportSettings *export_settings;
 };
 
 #endif