From 337cb4838dab83c003e34447a28548c0095e58b4 Mon Sep 17 00:00:00 2001
From: itsmattkc <34096995+itsmattkc@users.noreply.github.com>
Date: Mon, 27 Sep 2021 14:58:45 -0700
Subject: [PATCH] viewer: use clamp

Cleaner code
---
 app/widget/viewer/viewer.cpp | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/app/widget/viewer/viewer.cpp b/app/widget/viewer/viewer.cpp
index a114099c8..3eb589b37 100644
--- a/app/widget/viewer/viewer.cpp
+++ b/app/widget/viewer/viewer.cpp
@@ -31,6 +31,7 @@
 #include <QVBoxLayout>
 
 #include "audio/audiomanager.h"
+#include "common/clamp.h"
 #include "common/power.h"
 #include "common/ratiodialog.h"
 #include "common/timecodefunctions.h"
@@ -423,14 +424,8 @@ void ViewerWidget::QueueNextAudioBuffer()
   // NOTE: Hardcoded 2 second interval
   rational queue_end = audio_playback_queue_time_ + (2 * playback_speed_);
 
-  if (playback_speed_ < 0) {
-    // Limit to 0 if playing in reverse
-    queue_end = qMax(rational(0), queue_end);
-  } else {
-    // Limit to audio length if playing forwards
-    queue_end = qMin(GetConnectedNode()->GetAudioLength(), queue_end);
-  }
-
+  // Clamp queue end by zero and the audio length
+  queue_end  = clamp(queue_end, rational(0), GetConnectedNode()->GetAudioLength());
   if (queue_end == audio_playback_queue_time_) {
     // This will queue nothing, so stop the loop here
     return;