Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-output: part 1 - internal functions to cycle between outputs #163

Merged
merged 5 commits into from
Nov 6, 2022

Conversation

lack
Copy link
Collaborator

@lack lack commented Oct 11, 2022

In support of multi-output displays (#112), this changeset does some internal refactoring to support switching capture and pointer input to other outputs while a client is connected.

I have read and understood CONTRIBUTING.md and its associated documents.

@lack lack force-pushed the output_switching branch from d639577 to a4c65d6 Compare October 11, 2022 15:18
@lack lack changed the title Multi-output: keybind to cycle between outputs Multi-output: part 1 - internal functions to cycle between outputs Oct 11, 2022
src/main.c Outdated Show resolved Hide resolved
src/main.c Outdated Show resolved Hide resolved
src/output.c Outdated Show resolved Hide resolved
src/output.c Outdated Show resolved Hide resolved
lack added 5 commits October 11, 2022 20:14
Additionally, make the pointer setup reentrant, destroying the previous
setup if needed.

Signed-off-by: Jim Ramsay <[email protected]>
One to switch to an arbitrary outout, and then two helpers to switch
to the next/previous in the output list.

Signed-off-by: Jim Ramsay <[email protected]>
@lack lack force-pushed the output_switching branch from a4c65d6 to e414aa8 Compare October 12, 2022 00:47
Copy link
Owner

@any1 any1 left a comment

Choose a reason for hiding this comment

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

LGTM. I'll merge this with part 2.

iter->next : iter->prev;
}
struct output* output;
return wl_container_of(iter, output, link);
Copy link
Owner

Choose a reason for hiding this comment

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

I think you can just as well say wl_container_of(iter, struct output, link) here.

@any1 any1 merged commit cb95ce9 into any1:master Nov 6, 2022
@any1
Copy link
Owner

any1 commented Nov 6, 2022

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants