From 250cf2e7f07479eaf2a97a5cf3fa2def1d19dabc Mon Sep 17 00:00:00 2001 From: Alexander Golubev Date: Fri, 3 Jul 2026 01:46:23 +0300 Subject: kicker: fix incorrect operation priorities The ternary operator has lower priority than `-`, so `availableWidth` was always assigned `2` or `0` instead of subtracting the value from the width(). The mistake had no real consequence besides slightly less efficient drawing. Signed-off-by: Alexander Golubev --- kicker/taskbar/taskcontainer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kicker/taskbar/taskcontainer.cpp b/kicker/taskbar/taskcontainer.cpp index 50a1c1d52..338ae0195 100644 --- a/kicker/taskbar/taskcontainer.cpp +++ b/kicker/taskbar/taskcontainer.cpp @@ -810,8 +810,8 @@ void TaskContainer::drawButton(TQPainter *p) textPen = TQPen(colors.buttonText()); // textPen = p->pen(); } } - - int availableWidth = width() - (br.x() * 2) - textPos - 2 - (READ_MERGED_TASKBAR_SETTING(drawButtons) && KickerSettings::showDeepButtons())?2:0; + bool deepButtons = READ_MERGED_TASKBAR_SETTING(drawButtons) && KickerSettings::showDeepButtons(); + int availableWidth = width() - (br.x() * 2) - textPos - 2 - (deepButtons ? 2 : 0); if (m_filteredTasks.count() > 1) { availableWidth -= 8; -- cgit v1.2.3