From 7bdb2e79ed0c15d6d587fd170190890f5b9fdbb3 Mon Sep 17 00:00:00 2001 From: Patric Stout Date: Tue, 16 Feb 2021 20:37:58 +0100 Subject: Fix: framerate window showed a slightly higher rate than actually measured (#8682) The first point was counted, but also initialized as "last". As such, it didn't add to "total", but did add to "count", which made the "count" 1 more than the total actually represents. --- src/framerate_gui.cpp | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src/framerate_gui.cpp') diff --git a/src/framerate_gui.cpp b/src/framerate_gui.cpp index c324fde2e..0afb533f6 100644 --- a/src/framerate_gui.cpp +++ b/src/framerate_gui.cpp @@ -151,6 +151,10 @@ namespace { /* Total duration covered by collected points */ TimingMeasurement total = 0; + /* We have nothing to compare the first point against */ + point--; + if (point < 0) point = NUM_FRAMERATE_POINTS - 1; + while (point != last_point) { /* Only record valid data points, but pretend the gaps in measurements aren't there */ if (this->durations[point] != INVALID_DURATION) { -- cgit v1.2.3-54-g00ecf