From e17e6b84248119f878f34d246b68434b208aa8d5 Mon Sep 17 00:00:00 2001 From: Sebastian Englbrecht Date: Tue, 26 May 2026 21:03:58 +0200 Subject: DolphinView: store new zoom level in a variable after setZoomLevel() zoomLevel() called twice returns the same value to static analysis. Found with cppcheck --- src/views/dolphinview.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/views') diff --git a/src/views/dolphinview.cpp b/src/views/dolphinview.cpp index 3c973618c..4747b5ecc 100644 --- a/src/views/dolphinview.cpp +++ b/src/views/dolphinview.cpp @@ -504,12 +504,13 @@ void DolphinView::setZoomLevel(int level) const int oldZoomLevel = zoomLevel(); m_view->setZoomLevel(level); - if (zoomLevel() != oldZoomLevel) { + const int newZoomLevel = zoomLevel(); + if (newZoomLevel != oldZoomLevel) { ViewProperties props(viewPropertiesUrl()); props.setZoomLevel(level == m_defaultZoomLevel ? -1 : level); hideToolTip(); - Q_EMIT zoomLevelChanged(zoomLevel(), oldZoomLevel); + Q_EMIT zoomLevelChanged(newZoomLevel, oldZoomLevel); } } -- cgit v1.3.1