From e06f9174a1fa5c877ae32f0f6fe71ac5338f7fa4 Mon Sep 17 00:00:00 2001 From: thankyouverycool <66646555+thankyouverycool@users.noreply.github.com> Date: Sun, 11 Dec 2022 13:23:31 -0500 Subject: [PATCH] WindowServer: Remove unused bools in Menu::draw() Added in d522a6f and 1e604b7, their purpose snuffed out in 11bb88f like the faint pulse of a pleading candle, two lives of short excess, doomed to itemize their sins to no effect and for all eternity... --- Userland/Services/WindowServer/Menu.cpp | 9 --------- 1 file changed, 9 deletions(-) diff --git a/Userland/Services/WindowServer/Menu.cpp b/Userland/Services/WindowServer/Menu.cpp index fa481efad7..49646ef776 100644 --- a/Userland/Services/WindowServer/Menu.cpp +++ b/Userland/Services/WindowServer/Menu.cpp @@ -202,15 +202,6 @@ void Menu::draw() painter.draw_rect(rect, Color::Black); painter.fill_rect(rect.shrunken(2, 2), palette.menu_base()); - bool has_checkable_items = false; - bool has_items_with_icon = false; - for (auto& item : m_items) { - if (!item.is_visible()) - continue; - has_checkable_items = has_checkable_items | item.is_checkable(); - has_items_with_icon = has_items_with_icon | !!item.icon(); - } - // Draw the stripe first, which may extend outside of individual items. We can // skip this step when painting an individual item since we're drawing all of them painter.fill_rect(stripe_rect(), palette.menu_stripe());