SH-3405 FIX convert existing stats to lltrace system

final removal of remaining LLStat code

1 files changed, 98 insertions, 40 deletions

diff --git a/indra/llui/llstatbar.cpp b/indra/llui/llstatbar.cpp
index 535c6f96e3..6b40f8d475 100644
--- a/indra/llui/llstatbar.cpp
+++ b/indra/llui/llstatbar.cpp
@@ -34,7 +34,6 @@
 #include "llgl.h"
 #include "llfontgl.h"
 
-#include "llstat.h"
 #include "lluictrlfactory.h"
 #include "lltracerecording.h"
 
@@ -46,8 +45,10 @@ LLStatBar::LLStatBar(const Params& p)
 	  mUnitLabel(p.unit_label),
 	  mMinBar(p.bar_min),
 	  mMaxBar(p.bar_max),
-	  mStatp(LLStat::getInstance(p.stat)),
-	  mNewStatp(LLTrace::Count<>::getInstance(p.stat)),
+	  mCountFloatp(LLTrace::Count<>::getInstance(p.stat)),
+	  mCountIntp(LLTrace::Count<S64>::getInstance(p.stat)),
+	  mMeasurementFloatp(LLTrace::Measurement<>::getInstance(p.stat)),
+	  mMeasurementIntp(LLTrace::Measurement<S64>::getInstance(p.stat)),
 	  mTickSpacing(p.tick_spacing),
 	  mLabelSpacing(p.label_spacing),
 	  mPrecision(p.precision),
@@ -90,46 +91,62 @@ void LLStatBar::draw()
 		max = 0.f,
 		mean = 0.f;
 
-	if (mStatp)
+	LLTrace::PeriodicRecording& frame_recording = LLTrace::get_frame_recording();
+
+	if (mCountFloatp)
 	{
-		// Get the values.
+		LLTrace::Recording& last_frame_recording = frame_recording.getLastRecordingPeriod(); 
+
 		if (mPerSec)
 		{
-			current = mStatp->getCurrentPerSec();
-			min = mStatp->getMinPerSec();
-			max = mStatp->getMaxPerSec();
-			mean = mStatp->getMeanPerSec();
+			current = last_frame_recording.getPerSec(*mCountFloatp);
+			min = frame_recording.getPeriodMinPerSec(*mCountFloatp);
+			max = frame_recording.getPeriodMaxPerSec(*mCountFloatp);
+			mean = frame_recording.getPeriodMeanPerSec(*mCountFloatp);
 		}
 		else
 		{
-			current = mStatp->getCurrent();
-			min = mStatp->getMin();
-			max = mStatp->getMax();
-			mean = mStatp->getMean();
+			current = last_frame_recording.getSum(*mCountFloatp);
+			min = frame_recording.getPeriodMin(*mCountFloatp);
+			max = frame_recording.getPeriodMax(*mCountFloatp);
+			mean = frame_recording.getPeriodMean(*mCountFloatp);
 		}
 	}
-	else if (mNewStatp)
+	else if (mCountIntp)
 	{
-		LLTrace::PeriodicRecording& frame_recording = LLTrace::get_frame_recording();
 		LLTrace::Recording& last_frame_recording = frame_recording.getLastRecordingPeriod(); 
-		LLTrace::Recording& windowed_frame_recording = frame_recording.getTotalRecording(); 
 
 		if (mPerSec)
 		{
-			current = last_frame_recording.getPerSec(*mNewStatp);
-			//min = frame_window_recording.getMin(*mNewStatp) / frame_window_recording.getDuration();
-			//max = frame_window_recording.getMax(*mNewStatp) / frame_window_recording.getDuration();
-			mean = windowed_frame_recording.getPerSec(*mNewStatp);//frame_window_recording.getMean(*mNewStatp) / frame_window_recording.getDuration();
+			current = last_frame_recording.getPerSec(*mCountIntp);
+			min = frame_recording.getPeriodMinPerSec(*mCountIntp);
+			max = frame_recording.getPeriodMaxPerSec(*mCountIntp);
+			mean = frame_recording.getPeriodMeanPerSec(*mCountIntp);
 		}
 		else
 		{
-			current = last_frame_recording.getSum(*mNewStatp);
-			//min = last_frame_recording.getMin(*mNewStatp);
-			//max = last_frame_recording.getMax(*mNewStatp);
-			mean = windowed_frame_recording.getSum(*mNewStatp);
+			current = last_frame_recording.getSum(*mCountIntp);
+			min = frame_recording.getPeriodMin(*mCountIntp);
+			max = frame_recording.getPeriodMax(*mCountIntp);
+			mean = frame_recording.getPeriodMean(*mCountIntp);
 		}
 	}
-	
+	else if (mMeasurementFloatp)
+	{
+		LLTrace::Recording& recording = frame_recording.getTotalRecording();
+		current = recording.getLastValue(*mMeasurementFloatp);
+		min = recording.getMin(*mMeasurementFloatp);
+		max = recording.getMax(*mMeasurementFloatp);
+		mean = recording.getMean(*mMeasurementFloatp);
+	}
+	else if (mMeasurementIntp)
+	{
+		LLTrace::Recording& recording = frame_recording.getTotalRecording();
+		current = recording.getLastValue(*mMeasurementIntp);
+		min = recording.getMin(*mMeasurementIntp);
+		max = recording.getMax(*mMeasurementIntp);
+		mean = recording.getMean(*mMeasurementIntp);
+	}
 
 	if ((mUpdatesPerSec == 0.f) || (mUpdateTimer.getElapsedTimeF32() > 1.f/mUpdatesPerSec) || (mValue == 0.f))
 	{
@@ -176,7 +193,7 @@ void LLStatBar::draw()
 											 LLFontGL::RIGHT, LLFontGL::TOP);
 
 	value_format = llformat( "%%.%df", mPrecision);
-	if (mDisplayBar && mStatp)
+	if (mDisplayBar && (mCountFloatp || mCountIntp || mMeasurementFloatp || mMeasurementIntp))
 	{
 		std::string tick_label;
 
@@ -219,7 +236,7 @@ void LLStatBar::draw()
 		right = width;
 		gl_rect_2d(left, top, right, bottom, LLColor4(0.f, 0.f, 0.f, 0.25f));
 
-		if (mStatp->getNumValues() == 0)
+		if (frame_recording.getNumPeriods() == 0)
 		{
 			// No data, don't draw anything...
 			return;
@@ -236,26 +253,58 @@ void LLStatBar::draw()
 		right = (S32) ((max - mMinBar) * value_scale);
 		gl_rect_2d(left, top, right, bottom, LLColor4(1.f, 0.f, 0.f, 0.25f));
 
-		if (mDisplayHistory)
+		if (mDisplayHistory && (mCountFloatp || mCountIntp || mMeasurementFloatp || mMeasurementIntp))
 		{
-			S32 num_values = mStatp->getNumValues() - 1;
+			S32 num_values = frame_recording.getNumPeriods() - 1;
 			S32 i;
-			for (i = 0; i < num_values; i++)
+			for (i = 1; i <= num_values; i++)
 			{
-				if (i == mStatp->getNextBin())
-				{
-					continue;
-				}
 				if (mPerSec)
 				{
-					left = (S32)((mStatp->getPrevPerSec(i) - mMinBar) * value_scale);
-					right = (S32)((mStatp->getPrevPerSec(i) - mMinBar) * value_scale) + 1;
+					if (mCountFloatp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mCountFloatp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mCountFloatp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mCountIntp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mCountIntp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mCountIntp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mMeasurementFloatp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mMeasurementFloatp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mMeasurementFloatp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mMeasurementIntp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mMeasurementIntp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getPerSec(*mMeasurementIntp)  - mMinBar) * value_scale) + 1;
+					}
 					gl_rect_2d(left, bottom+i+1, right, bottom+i, LLColor4(1.f, 0.f, 0.f, 1.f));
 				}
 				else
 				{
-					left = (S32)((mStatp->getPrev(i) - mMinBar) * value_scale);
-					right = (S32)((mStatp->getPrev(i) - mMinBar) * value_scale) + 1;
+					if (mCountFloatp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mCountFloatp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mCountFloatp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mCountIntp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mCountIntp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mCountIntp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mMeasurementFloatp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mMeasurementFloatp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mMeasurementFloatp)  - mMinBar) * value_scale) + 1;
+					}
+					else if (mMeasurementIntp)
+					{
+						left = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mMeasurementIntp)  - mMinBar) * value_scale);
+						right = (S32)((frame_recording.getPrevRecordingPeriod(i).getSum(*mMeasurementIntp)  - mMinBar) * value_scale) + 1;
+					}
 					gl_rect_2d(left, bottom+i+1, right, bottom+i, LLColor4(1.f, 0.f, 0.f, 1.f));
 				}
 			}
@@ -279,6 +328,15 @@ void LLStatBar::draw()
 	LLView::draw();
 }
 
+void LLStatBar::setStat(const std::string& stat_name)
+{
+	mCountFloatp = LLTrace::Count<>::getInstance(stat_name);
+	mCountIntp = LLTrace::Count<S64>::getInstance(stat_name);
+	mMeasurementFloatp = LLTrace::Measurement<>::getInstance(stat_name);
+	mMeasurementIntp = LLTrace::Measurement<S64>::getInstance(stat_name);
+}
+
+
 void LLStatBar::setRange(F32 bar_min, F32 bar_max, F32 tick_spacing, F32 label_spacing)
 {
 	mMinBar = bar_min;
@@ -293,9 +351,9 @@ LLRect LLStatBar::getRequiredRect()
 
 	if (mDisplayBar)
 	{
-		if (mDisplayHistory && mStatp)
+		if (mDisplayHistory)
 		{
-			rect.mTop = 35 + mStatp->getNumBins();
+			rect.mTop = 35 + LLTrace::get_frame_recording().getNumPeriods();
 		}
 		else
 		{