From 63ac6ced31d7bbbe31552b73f1108715352e6970 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Thu, 2 Feb 2023 14:18:05 +0100 Subject: [PATCH] LibGUI: Merge two loops over the segments in Breadcrummbar::relayout() --- Userland/Libraries/LibGUI/Breadcrumbbar.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Userland/Libraries/LibGUI/Breadcrumbbar.cpp b/Userland/Libraries/LibGUI/Breadcrumbbar.cpp index 632daccfcd..343b8b60e9 100644 --- a/Userland/Libraries/LibGUI/Breadcrumbbar.cpp +++ b/Userland/Libraries/LibGUI/Breadcrumbbar.cpp @@ -171,6 +171,7 @@ void Breadcrumbbar::did_change_font() void Breadcrumbbar::relayout() { + auto total_width = 0; for (auto& segment : m_segments) { VERIFY(segment.button); auto& button = *segment.button; @@ -185,12 +186,11 @@ void Breadcrumbbar::relayout() button.set_max_size(segment.width, 16 + 8); button.set_min_size(segment.shrunken_width, 16 + 8); + + total_width += segment.width; } - auto remaining_width = 0; - - for (auto& segment : m_segments) - remaining_width += segment.width; + auto remaining_width = total_width; for (auto& segment : m_segments) { if (remaining_width > width() && !segment.button->is_checked()) {