From c8e750602793436447dd065acf0c1cfed11cf2e2 Mon Sep 17 00:00:00 2001
From: Harshit Maurya <hmaurya999@gmail.com>
Date: Sat, 16 May 2020 01:48:12 +0530
Subject: convert to const methods

---
 src/waveform/renderers/waveformmark.cpp           | 4 ++--
 src/waveform/renderers/waveformmark.h             | 2 +-
 src/waveform/renderers/waveformwidgetrenderer.cpp | 5 +++--
 src/waveform/renderers/waveformwidgetrenderer.h   | 2 +-
 4 files changed, 7 insertions(+), 6 deletions(-)

(limited to 'src/waveform/renderers')

diff --git a/src/waveform/renderers/waveformmark.cpp b/src/waveform/renderers/waveformmark.cpp
index 81a3d1cbe5..b9449d24b4 100644
--- a/src/waveform/renderers/waveformmark.cpp
+++ b/src/waveform/renderers/waveformmark.cpp
@@ -105,7 +105,7 @@ void WaveformMark::setBaseColor(QColor baseColor) {
     m_labelColor = Color::chooseColorByBrightness(baseColor, QColor(255,255,255,255), QColor(0,0,0,255));
 };
 
-bool WaveformMark::contains(QPoint point, Qt::Orientation orientation) {
+bool WaveformMark::contains(QPoint point, Qt::Orientation orientation) const {
     // Without some padding, the user would only have a single pixel width that
     // would count as hovering over the WaveformMark.
     float lineHoverPadding = 5.0;
@@ -118,5 +118,5 @@ bool WaveformMark::contains(QPoint point, Qt::Orientation orientation) {
     bool lineHovered = m_linePosition >= position - lineHoverPadding &&
             m_linePosition <= position + lineHoverPadding;
 
-    return (m_label.area().contains(point) || lineHovered);
+    return m_label.area().contains(point) || lineHovered;
 }
diff --git a/src/waveform/renderers/waveformmark.h b/src/waveform/renderers/waveformmark.h
index 5e3150126a..abb0da8286 100644
--- a/src/waveform/renderers/waveformmark.h
+++ b/src/waveform/renderers/waveformmark.h
@@ -67,7 +67,7 @@ class WaveformMark {
     }
 
     // Check if a point (in image co-ordinates) lies on drawn image.
-    bool contains(QPoint point, Qt::Orientation orientation);
+    bool contains(QPoint point, Qt::Orientation orientation) const;
 
     QColor m_textColor;
     QString m_text;
diff --git a/src/waveform/renderers/waveformwidgetrenderer.cpp b/src/waveform/renderers/waveformwidgetrenderer.cpp
index c73cc623d5..bd181fdf0f 100644
--- a/src/waveform/renderers/waveformwidgetrenderer.cpp
+++ b/src/waveform/renderers/waveformwidgetrenderer.cpp
@@ -289,13 +289,14 @@ void WaveformWidgetRenderer::setTrack(TrackPointer track) {
     }
 }
 
-WaveformMarkPointer WaveformWidgetRenderer::getCueMarkAtPoint(QPoint point) {
+WaveformMarkPointer WaveformWidgetRenderer::getCueMarkAtPoint(QPoint point) const {
     for (const auto& pMark : m_markPositions.keys()) {
         int markImagePositionInWidgetSpace = m_markPositions[pMark];
         QPoint pointInImageSpace;
         if (getOrientation() == Qt::Horizontal) {
             pointInImageSpace = QPoint(point.x() - markImagePositionInWidgetSpace, point.y());
-        } else { /* Vertical */
+        } else {
+            DEBUG_ASSERT(getOrientation() == Qt::Vertical);
             pointInImageSpace = QPoint(point.x(), point.y() - markImagePositionInWidgetSpace);
         }
         if (pMark->contains(pointInImageSpace, getOrientation())) {
diff --git a/src/waveform/renderers/waveformwidgetrenderer.h b/src/waveform/renderers/waveformwidgetrenderer.h
index a255eaae65..6e7ef4b042 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 mark at a point on the waveform widget.
-    WaveformMarkPointer getCueMarkAtPoint(QPoint point);
+    WaveformMarkPointer getCueMarkAtPoint(QPoint point) const;
 
     double getFirstDisplayedPosition() const { return m_firstDisplayedPosition;}
     double getLastDisplayedPosition() const { return m_lastDisplayedPosition;}
-- 
cgit v1.2.3