Skip to content

Remove factory methods in Display

Vlad Zahorodnii requested to merge work/remove-factory-methods into master

The main reason why we have factory methods is that up to some point, kwayland had its own signal to indicate when globals have to be removed.

Now that all globals add destroy listeners for the wl_display object, we don't have that signal. Most factory methods are equivalent to doing new T(display).

Besides adding unnecessary boilerplate code, another reason to get rid of the factory methods is to reduce the amount of merge conflicts. If several persons work on implementing wayland protocols at the same time, sooner or later someone will have to resolve merge conflicts in Display.

Merge request reports