[vlc-devel] [PATCH 1/2] qt: fix missing return value in ChapterModel

Abel Tesfaye abeltesfaye45 at gmail.com
Fri Jun 14 17:44:46 CEST 2019


From: Abel Tesfaye <Abeltesfaye45 at gmail.com>

---
 modules/gui/qt/util/input_models.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/modules/gui/qt/util/input_models.cpp b/modules/gui/qt/util/input_models.cpp
index 918f93193d..125586a60e 100644
--- a/modules/gui/qt/util/input_models.cpp
+++ b/modules/gui/qt/util/input_models.cpp
@@ -346,8 +346,7 @@ void ChapterListModel::resetTitle(const vlc_player_title *newTitle)
 
 QString ChapterListModel::getNameAtPosition(float pos) const
 {
-    if(m_title == nullptr)
-        return qtr("nothing found");
+    if(m_title != nullptr){
 
     vlc_tick_t posTime = pos * m_title->length;
     int prevChapterIndex = 0;
@@ -357,13 +356,15 @@ QString ChapterListModel::getNameAtPosition(float pos) const
         vlc_tick_t currentChapterTime = m_title->chapters[i].time;
 
         if(currentChapterTime > posTime)
-           return qfu(m_title->chapters[prevChapterIndex].name);
+            return qfu(m_title->chapters[prevChapterIndex].name);
 
         else if(i == (m_title->chapter_count - 1))
             return qfu(m_title->chapters[i].name);
 
         prevChapterIndex = i;
     }
+    }
+    return QString();
 }
 
 //***************************
-- 
2.21.0



More information about the vlc-devel mailing list