Improve CRippleManager::AddRipple

Former-commit-id: 5b4dfc0258

include/MetroidPrime/CRippleManager.hpp

@@ -20,6 +20,15 @@ public:
   void Update(float dt);
   float GetLastRippleDeltaTime(TUniqueId rippler) const;
   void AddRipple(const CRipple& ripple);
+
+  void SetMaxTimeFalloff(float time) { x0_maxTimeFalloff = time; }
+  float GetMaxTimeFalloff() const { return x0_maxTimeFalloff; }
+
+  rstl::vector<CRipple>& Ripples() { return x4_ripples; }
+  const rstl::vector<CRipple>& GetRipples() const { return x4_ripples; }
+  
+  void SetAlpha(float a) { x14_alpha = a; }
+  float GetAlpha() const { return x14_alpha; }
 };
 
 #endif // _CRIPPLEMANAGER

src/MetroidPrime/CRippleManager.cpp

@@ -42,22 +42,23 @@ float CRippleManager::GetLastRippleDeltaTime(TUniqueId rippler) const {
 
 void CRippleManager::AddRipple(const CRipple& ripple) {
   float maxTime = 0.f;
-  float itTime;
-  rstl::vector< CRipple >::iterator t;
   rstl::vector< CRipple >::iterator oldestRipple = x4_ripples.end();
-  rstl::vector< CRipple >::iterator it = x4_ripples.begin();
 
-  for (; (t = oldestRipple,
+  for (rstl::vector< CRipple >::iterator it = x4_ripples.begin(); it != x4_ripples.end(); ++it) {
-         it != x4_ripples.end() && (itTime = it->GetTime(), t = it, itTime != 9999.0f));
+    if (it->GetTime() == 9999.0f) {
-       ++it) {
-    if (itTime > maxTime) {
       oldestRipple = it;
-      maxTime = itTime;
+      break;
+    }
+
+    if (it->GetTime() > maxTime) {
+      oldestRipple = it;
+      maxTime = it->GetTime();
     }
   }
 
-  if (t != x4_ripples.end()) {
+  if (oldestRipple != x4_ripples.end()) {
-    *t = ripple;
+    *oldestRipple = ripple;
-    x0_maxTimeFalloff = rstl::max_val(x0_maxTimeFalloff, ripple.GetTimeFalloff());
+    oldestRipple->SetTime(0.f);
+    SetMaxTimeFalloff(rstl::max_val(GetMaxTimeFalloff(), ripple.GetTimeFalloff()));
   }
 }