[FIXED] LLVisualEffectParams::step(bool isLast) will always be called with...

[FIXED] LLVisualEffectParams::step(bool isLast) will always be called with isLast == false (fixes RLVa spheres issues when AA is disabled)

indra/newview/llvisualeffect.cpp

@@ -114,11 +114,12 @@ void LLVfxManager::runEffect(EVisualEffect eCode, LLVisualEffectParams* pParams)
 
 	auto itEffect = boost::make_filter_iterator(pred, m_Effects.begin(), m_Effects.end()),
 	     endEffect = boost::make_filter_iterator(pred, m_Effects.end(), m_Effects.end());
-	for (; itEffect != endEffect; ++itEffect)
+	while (itEffect != endEffect)
 	{
+		LLVisualEffect* pEffect = *itEffect;
 		if (pParams)
-			pParams->step(itEffect == endEffect);
-		(*itEffect)->run(pParams);
+			pParams->step(++itEffect == endEffect);
+		pEffect->run(pParams);
 	}
 }
 

indra/newview/llvisualeffect.h

@@ -163,6 +163,7 @@ class LLVfxManager : public LLSingleton<LLVfxManager>
 	template<typename T> T* getEffect(const LLUUID& idEffect) const { return dynamic_cast<T*>(getEffect(idEffect)); }
 	LLVisualEffect* getEffect(EVisualEffect eCode) const;
 	template<typename T> T* getEffect(EVisualEffect eCode) const { return dynamic_cast<T*>(getEffect(eCode)); }
+	bool            hasEffect(EVisualEffect eCode) const         { return getEffect(eCode); }
 	bool            removeEffect(const LLUUID& idEffect);
 	void            runEffect(EVisualEffect eCode, LLVisualEffectParams* pParams = nullptr);
 	void            runEffect(EVisualEffectType eType, LLVisualEffectParams* pParams = nullptr);