From df861fb33def53340503dd93e1ce4bf7764ab53f Mon Sep 17 00:00:00 2001
From: itsmattkc <itsmattkc@gmail.com>
Date: Thu, 29 Apr 2021 11:26:11 +1000
Subject: [PATCH] viewer: if only video stream is still, don't use as rate in
 project explorer

Minor UI improvement
---
 app/node/output/viewer/viewer.cpp | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/app/node/output/viewer/viewer.cpp b/app/node/output/viewer/viewer.cpp
index bc9869117..c1c8715f4 100644
--- a/app/node/output/viewer/viewer.cpp
+++ b/app/node/output/viewer/viewer.cpp
@@ -121,13 +121,12 @@ QString ViewerOutput::duration() const
 
 QString ViewerOutput::rate() const
 {
-  if (HasEnabledVideoStreams()) {
-    // This is a video editor, prioritize video streams
-    VideoParams video_stream = GetFirstEnabledVideoStream();
+  VideoParams video_stream;
 
-    if (video_stream.video_type() != VideoParams::kVideoTypeStill) {
-      return tr("%1 FPS").arg(video_stream.frame_rate().toDouble());
-    }
+  if (HasEnabledVideoStreams()
+      && (video_stream = GetFirstEnabledVideoStream()).video_type() != VideoParams::kVideoTypeStill) {
+    // This is a video editor, prioritize video streams
+    return tr("%1 FPS").arg(video_stream.frame_rate().toDouble());
   } else if (HasEnabledAudioStreams()) {
     // No video streams, return audio
     AudioParams audio_stream = GetFirstEnabledAudioStream();