Sivecano/dwl - Change L2ERA3TQPLMFAEQVNC5RFDTO532JQPHLYVBCE3JZT2J6VSXG4IJQC

Fix crash when disabling focused mon

m->link.next leads to errors if the monitor to disable doesn't have a "next" (right) monitor and is currently focused. dirtmon() does more checks.

In some previous commits m->link.next is told to be left monitor, which is wrong

Also focusclient() explicitly checks for disabled monitors (this fixes in case of more than one disabled monitor)

Created by Stivvo on October 31, 2020

L2ERA3TQPLMFAEQVNC5RFDTO532JQPHLYVBCE3JZT2J6VSXG4IJQC

Dependencies

In channels

main

Change contents

Insertion in dwl.c at line 1462 [4.4]
[3.806]
[3.806]
```
	Arg ar = {.i = -1};
```

Insertion in dwl.c at line 1468 [4.4]

[3.924]

		if (!config_head->state.enabled) {
			wl_list_for_each(m, &mons, link) {
				if (m->wlr_output->name == wlr_output->name) {
					// make sure that the monitor to clean is focused
					selmon = m;
					focusclient(selclient(), focustop(selmon), 1);
					// focus the left monitor (relative to the current focus)
					focusmon(&ar);
					closemon(m, wl_container_of(&selmon->link, newmon, link));
				}
			}
		}

Deletion in dwl.c at line 1482 [4.4]

B:BD[3.985] → [5.3:123]

B:BD[5.123] → [2.27:90]

		if (!config_head->state.enabled)
			wl_list_for_each(m, &mons, link)
				if (m->wlr_output->name == wlr_output->name)
					closemon(m, wl_container_of(m->link.next, newmon, link));