Decompile fn_17510 (C3dActor::CopyVolumes)

Batch #12528 by cremane: 0 byte-identical, 1 confirmed by peer review.

- fn_17510 C3dActor::CopyVolumes: logic confirmed by peer review

Peer-reviewed-by: Yommy, gordaoprogramas, dori
Accepted without human review: peers confirmed the logic ("same" ahead by at least 3, with 70% of the votes).

Source/Contents/Models/ModelActor.cpp

@@ -220,8 +220,10 @@ C3dNode* C3dActorV2::Find(const char* name) {
 // OFFSET: 0x00AB94D0
 // FUNCTION: fn_17510
 void C3dActor::CopyVolumes(const std::list<ModelVolume*>& source) {
-  for (const ModelVolume* volume : source) {
-    this->volumes.push_back(new ModelVolume(*volume));
+  for (auto it = source.begin(); it != source.end(); ++it) {
+    ModelVolume* volume = new ModelVolume;
+    *volume = **it;
+    this->volumes.push_back(volume);
   }
 }
 

Source/Contents/Models/ModelActor.h

@@ -22,6 +22,7 @@ public:
   virtual void MarkOccluded(const float* ray);
   virtual void SetSpeed(float speed);
   void CopyVolumes(const std::list<ModelVolume*>& source);
+  void Fade();
 
   /* 0004 */ float position[3];
   /* 0010 */ float rotation[3];
@@ -50,7 +51,6 @@ public:
 
 protected:
   void FinishBounds(const float* bounds, bool modern);
-  void Fade();
   bool Reversed() const;
 };