waveformwidgetrenderer: remove std::optional wrapping

2 files changed, 6 insertions, 11 deletions

diff --git a/src/waveform/renderers/waveformwidgetrenderer.cpp b/src/waveform/renderers/waveformwidgetrenderer.cpp
index ce6b9b5cd7..79499d0712 100644
--- a/src/waveform/renderers/waveformwidgetrenderer.cpp
+++ b/src/waveform/renderers/waveformwidgetrenderer.cpp
@@ -289,7 +289,7 @@ void WaveformWidgetRenderer::setTrack(TrackPointer track) {
     }
 }
 
-std::optional<CuePointer> WaveformWidgetRenderer::getCueAtPoint(QPoint point) {
+CuePointer WaveformWidgetRenderer::getCueAtPoint(QPoint point) {
     WaveformMarkPointer pSelectedMark;
     const int lineHoverpadding = 5;
     for (const auto& pMark : m_markLabelOffsets.keys()) {
@@ -333,19 +333,14 @@ std::optional<CuePointer> WaveformWidgetRenderer::getCueAtPoint(QPoint point) {
         }
     }
     if (!pSelectedMark) {
-        return std::nullopt;
+        return static_cast<CuePointer>(nullptr);
     }
-    CuePointer pSelectedCue;
+
     QList<CuePointer> cueList = getTrackInfo()->getCuePoints();
     for (const auto& pCue : cueList) {
         if (pCue->getHotCue() == pSelectedMark->getHotCue()) {
-            pSelectedCue = pCue;
-            break;
+            return pCue;
         }
     }
-    if (pSelectedCue != nullptr) {
-        return pSelectedCue;
-    }
-
-    return std::nullopt;
+    return static_cast<CuePointer>(nullptr);
 }
diff --git a/src/waveform/renderers/waveformwidgetrenderer.h b/src/waveform/renderers/waveformwidgetrenderer.h
index 4b67dc599b..c4841c091b 100644
--- a/src/waveform/renderers/waveformwidgetrenderer.h
+++ b/src/waveform/renderers/waveformwidgetrenderer.h
@@ -41,7 +41,7 @@ class WaveformWidgetRenderer {
     inline const char* getGroup() const { return m_group;}
     const TrackPointer getTrackInfo() const { return m_pTrack;}
     // Get cue at a point on the waveform widget.
-    std::optional<CuePointer> getCueAtPoint(QPoint point);
+    CuePointer getCueAtPoint(QPoint point);
 
     double getFirstDisplayedPosition() const { return m_firstDisplayedPosition;}
     double getLastDisplayedPosition() const { return m_lastDisplayedPosition;}