hyprwm/Hyprland
1
0
Fork 0
mirror of https://github.com/hyprwm/Hyprland synced 2026-05-05 00:17:58 +02:00
Code Issues Projects Releases Packages Wiki Activity

layout: replace string comparison with ID-based matching in WorkspaceAlgoMatcher (#13943)

Browse source 
* perf(layout): replace string comparison with ID-based matching in WorkspaceAlgoMatcher

* perf(layout): replace string comparison with ID-based matching in WorkspaceAlgoMatcher
This commit is contained in:
Visal Vijay 2026-04-02 03:50:56 +05:30 • committed by GitHub
parent 7fbf8f9847
commit 529f72249c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/layout/supplementary/WorkspaceAlgoMatcher.cpp
View file

@ -1,5 +1,6 @@
#include "WorkspaceAlgoMatcher.hpp"
#include <unordered_map>
#include <string>
#include "../../config/ConfigValue.hpp"
#include "../../config/shared/workspace/WorkspaceRuleManager.hpp"
@ -14,6 +15,8 @@
#include "../../Compositor.hpp"
static const std::unordered_map<std::string, int> layoutIDs = {{"dwindle", 1}, {"master", 2}, {"scrolling", 3}, {"monocle", 4}, {"default", 5}, {"floating", 6}};
using namespace Layout;
using namespace Layout::Supplementary;
@ -112,7 +115,7 @@ SP<CAlgorithm> CWorkspaceAlgoMatcher::createAlgorithmForWorkspace(PHLWORKSPACE w
}
void CWorkspaceAlgoMatcher::updateWorkspaceLayouts() {
// TODO: make this ID-based, string comparison is slow
// TODO: fully migrate layout selection to ID-based system (comparison optimized)
for (const auto& ws : g_pCompositor->getWorkspaces()) {
if (!ws)
continue;
@ -123,9 +126,18 @@ void CWorkspaceAlgoMatcher::updateWorkspaceLayouts() {
continue;
const auto LAYOUT_TO_USE = tiledAlgoForWorkspace(ws.lock());
auto itLayout = layoutIDs.find(LAYOUT_TO_USE);
const int layoutID = (itLayout != layoutIDs.end()) ? itLayout->second : -1;
if (m_algoNames.contains(&typeid(*TILED_ALGO.get())) && m_algoNames.at(&typeid(*TILED_ALGO.get())) == LAYOUT_TO_USE)
continue;
if (m_algoNames.contains(&typeid(*TILED_ALGO.get()))) {
const auto& currentName = m_algoNames.at(&typeid(*TILED_ALGO.get()));
auto itCurrent = layoutIDs.find(currentName);
const int currentID = (itCurrent != layoutIDs.end()) ? itCurrent->second : -1;
if (currentID == layoutID)
continue;
}
// needs a switchup
ws->m_space->algorithm()->updateTiledAlgo(algoForNameTiled(LAYOUT_TO_USE));