Skip to content

Removing the code that deletes an existing monitor in RRMonitorAdd

Michael Wyraz requested to merge mwyraz/xserver:server-21.1-pr1 into server-21.1-branch

This is a backport of !981 (merged) to 21.1

In commit 7e1f86d4 monitor support was added to randr. At this time it seemed to be reasonable not to have more than one (virtual) monitor on a particular physical display. The code was never changed since.

Nowadays, extremely large displays exists (4k displays, ultra-wide displays). In some use cases it makes sense to split these large physical displays into multiple virtual monitors. An example are ultra-wide screens that can be split into 2 monitors. The change in this commit makes this work.

Besides that, removing a monitor in a function that is called "RRMonitorAdd" is bad practice and causes unexpected behaviour.

Merge request reports