From 3c88ac8ead1fec0fb76cf8a48da0b8e1afe864df Mon Sep 17 00:00:00 2001 From: Vaxry Date: Sat, 4 Apr 2026 23:20:32 +0100 Subject: desktop/window: optimize getRealBorderSize() --- src/desktop/view/Window.cpp | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) (limited to 'src/desktop/view/Window.cpp') diff --git a/src/desktop/view/Window.cpp b/src/desktop/view/Window.cpp index e9f22021..4506d5fb 100644 --- a/src/desktop/view/Window.cpp +++ b/src/desktop/view/Window.cpp @@ -830,15 +830,25 @@ void CWindow::updateWindowData(const Config::CWorkspaceRule& workspaceRule) { m_ruleApplicator->rounding().matchOptional(workspaceRule.m_noRounding.value_or(false) ? std::optional(0) : std::nullopt, Desktop::Types::PRIORITY_WORKSPACE_RULE); m_ruleApplicator->noShadow().matchOptional(workspaceRule.m_noShadow, Desktop::Types::PRIORITY_WORKSPACE_RULE); + + m_borderSizeCacheDirty = true; } int CWindow::getRealBorderSize() const { - if ((m_workspace && isEffectiveInternalFSMode(FSMODE_FULLSCREEN)) || !m_ruleApplicator->decorate().valueOrDefault()) + if (!m_borderSizeCacheDirty) + return m_cachedBorderSize; + + if ((m_workspace && isEffectiveInternalFSMode(FSMODE_FULLSCREEN)) || !m_ruleApplicator->decorate().valueOrDefault()) { + m_cachedBorderSize = 0; + m_borderSizeCacheDirty = false; return 0; + } static auto PBORDERSIZE = CConfigValue("general:border_size"); - return m_ruleApplicator->borderSize().valueOr(*PBORDERSIZE); + m_cachedBorderSize = m_ruleApplicator->borderSize().valueOr(*PBORDERSIZE); + m_borderSizeCacheDirty = false; + return m_cachedBorderSize; } float CWindow::getScrollMouse() { @@ -1533,6 +1543,8 @@ Vector2D CWindow::getReportedSize() { } void CWindow::updateDecorationValues() { + m_borderSizeCacheDirty = true; + static auto PACTIVECOL = CConfigValue("general:col.active_border"); static auto PINACTIVECOL = CConfigValue("general:col.inactive_border"); static auto PNOGROUPACTIVECOL = CConfigValue("general:col.nogroup_border_active"); -- cgit v1.2.3