Skip to content
This repository was archived by the owner on Nov 1, 2021. It is now read-only.

Comments

Minor surface refactor#3169

Merged
emersion merged 7 commits intoswaywm:masterfrom
vyivel:surface-refactor-smol
Sep 6, 2021
Merged

Minor surface refactor#3169
emersion merged 7 commits intoswaywm:masterfrom
vyivel:surface-refactor-smol

Conversation

@vyivel
Copy link
Member

@vyivel vyivel commented Sep 6, 2021

A bunch of commits from #3143, including #3120 and #3122.

cc @emersion

Changes

  • wlr_subsurface.parent_link is removed

Instead, use wlr_subsurface.current.link; same goes for wlr_subsurface.parent_pending_link.

  • wlr_surface.subsurfaces_{above,below} is moved to wlr_surface_state

Use wlr_surface.current.subsurfaces_{above,below}.

Kirill Primak and others added 7 commits September 6, 2021 18:07
Caching frame callback lists is actually the correct behavior, because
if a surface is locked because of e.g. subsurface synchronization,
clients would expect to receive frame done events only after the
pending state is actually committed.
Move the wlr_subsurface parent link to the subsurface state.

This is a dumb find/replace operation. This shouldn't result in
any behavior change.
This makes the naming consistent with other resource implementations.
Copy link
Member

@emersion emersion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@emersion emersion merged commit 610f0c0 into swaywm:master Sep 6, 2021
@emersion emersion added the breaking Breaking change in public API label Sep 6, 2021
@vyivel vyivel deleted the surface-refactor-smol branch September 20, 2021 06:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

breaking Breaking change in public API

Development

Successfully merging this pull request may close these issues.

2 participants