diff options
| author | Dickby <felix.dick@web.de> | 2022-09-23 19:13:05 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-23 19:13:05 +0200 |
| commit | bccc81d306dd35cca2eeac2897e19064c7b3d24f (patch) | |
| tree | e46a76eb5566c148611b529d8f332e01faf73e9b /src/layout/DwindleLayout.cpp | |
| parent | 718de0d9fa6baa4fcfe0ee1c0a5ad8a36d5835ec (diff) | |
Fixing my last change
It wasn't a problem with the bug i tried to fix.
But there would be a problem if some function would dereference pPreviousGroupMember from the node that was PHEAD->pNextGroupMember.
Please don't be mean!
Diffstat (limited to 'src/layout/DwindleLayout.cpp')
| -rw-r--r-- | src/layout/DwindleLayout.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/layout/DwindleLayout.cpp b/src/layout/DwindleLayout.cpp index 26536728..59b36156 100644 --- a/src/layout/DwindleLayout.cpp +++ b/src/layout/DwindleLayout.cpp @@ -821,15 +821,14 @@ void CHyprDwindleLayout::toggleWindowGroup(CWindow* pWindow) { PWINDOW->m_bHidden = false; } - if (PHEAD->pPreviousGroupMember) { + if (PHEAD->pPreviousGroupMember) PHEAD->pPreviousGroupMember->pNextGroupMember = PHEAD->pNextGroupMember; - PHEAD->pPreviousGroupMember = nullptr; - } - if (PHEAD->pNextGroupMember) { + if (PHEAD->pNextGroupMember) PHEAD->pNextGroupMember->pPreviousGroupMember = PHEAD->pPreviousGroupMember; - PHEAD->pNextGroupMember = nullptr; - } + + PHEAD->pPreviousGroupMember = nullptr; + PHEAD->pNextGroupMember = nullptr; onWindowRemoved(PHEAD->pWindow); |