From 6841b6d9bb78af28c4d89c936c5663b31aa4fd5a Mon Sep 17 00:00:00 2001 From: Jan Holthuis Date: Thu, 19 Nov 2020 22:58:41 +0100 Subject: Do not return void expressions Fixes a bunch of `-Wclazy-returning-void-expression` warnings, since returning void expressions is confusing and may hide subtle bugs. See https://github.com/KDE/clazy/blob/master/docs/checks/README-returning-void-expression.md for details. --- src/library/browse/browsethread.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/library/browse') diff --git a/src/library/browse/browsethread.cpp b/src/library/browse/browsethread.cpp index e3badf5d3d..774e56f9fc 100644 --- a/src/library/browse/browsethread.cpp +++ b/src/library/browse/browsethread.cpp @@ -142,7 +142,8 @@ void BrowseThread::populateModel() { if (thisPath.dir() != newPath.dir()) { qDebug() << "Abort populateModel()"; - return populateModel(); + populateModel(); + return; } QList row_data; -- cgit v1.2.3