Don't recalculate lighting every frame. Fix altitude transitions. On track switch ignore time threshold on blender.

2 files changed, 12 insertions, 7 deletions

diff --git a/indra/newview/llenvironment.cpp b/indra/newview/llenvironment.cpp
index 938cf857ee..e3a030225d 100644
--- a/indra/newview/llenvironment.cpp
+++ b/indra/newview/llenvironment.cpp
@@ -164,7 +164,7 @@ namespace
                 return;
             }
 
-            mTrackTransitionStart = mTarget->buildDerivedClone();
+            LLSettingsBase::ptr_t pstartsetting = mTarget->buildDerivedClone();
             mTrackNo = use_trackno;
 
             LLSettingsBase::Seconds now = getAdjustedNow() + LLEnvironment::TRANSITION_ALTITUDE;
@@ -177,7 +177,11 @@ namespace
             LLSettingsBase::BlendFactor blendf = calculateBlend(targetpos, targetspan);
             pendsetting->blend((*bounds.second).second, blendf);
 
-            reset(mTrackTransitionStart, pendsetting, LLEnvironment::TRANSITION_ALTITUDE);
+//          pstartsetting->setValue(LLSettingsSky::SETTING_AMBIENT, LLColor3(0.0, 1.0, 0.0));
+//          pendsetting->setValue(LLSettingsSky::SETTING_AMBIENT, LLColor3(1.0, 0.0, 0.0));
+
+            setIgnoreTimeDeltaThreshold(true); // for the next span ignore the time delta threshold.
+            reset(pstartsetting, pendsetting, LLEnvironment::TRANSITION_ALTITUDE);
         }
 
     protected:
@@ -185,12 +189,12 @@ namespace
         {
             if (trackno == 0)
             {   // We are dealing with the water track.  There is only ever one.
-                return 0;
+                return trackno;
             }
 
-            for (S32 test = trackno; test == 0; --test)
+            for (S32 test = trackno; test != 0; --test)
             {   // Find the track below the requested one with data.
-                LLSettingsDay::CycleTrack_t &track = mDay->getCycleTrack(mTrackNo);
+                LLSettingsDay::CycleTrack_t &track = mDay->getCycleTrack(test);
 
                 if (!track.empty())
                     return test;
@@ -226,7 +230,6 @@ namespace
         S32                         mTrackNo;
         LLSettingsBase::Seconds     mCycleLength;
         LLSettingsBase::Seconds     mCycleOffset;
-        LLSettingsBase::ptr_t       mTrackTransitionStart;
 
         void onFinishedSpan()
         {
diff --git a/indra/newview/llvosky.cpp b/indra/newview/llvosky.cpp
index 52b5b1a3a2..7c1fda46ff 100644
--- a/indra/newview/llvosky.cpp
+++ b/indra/newview/llvosky.cpp
@@ -633,12 +633,14 @@ bool LLVOSky::updateSky()
 		direction.normalize();
 		const F32 dot_lighting = direction * mLastLightingDirection;
 
+        //_WARNS("LAPRAS") << " <" << direction.getValue() << "> dot <" << mLastLightingDirection << "> = " << dot_lighting << " (threshold is " << LIGHT_DIRECTION_THRESHOLD << ")" << LL_ENDL;
+
 		LLColor3 delta_color;
 		delta_color.setVec(mLastTotalAmbient.mV[0] - total_ambient.mV[0],
 							mLastTotalAmbient.mV[1] - total_ambient.mV[1],
                             mLastTotalAmbient.mV[2] - total_ambient.mV[2]);
 
-        bool light_direction_changed = (dot_lighting >= LIGHT_DIRECTION_THRESHOLD);
+        bool light_direction_changed = (dot_lighting < LIGHT_DIRECTION_THRESHOLD);
         bool color_changed = (delta_color.length() >= COLOR_CHANGE_THRESHOLD);
 
         mForceUpdate = mForceUpdate || light_direction_changed;