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

wayland: setup with new object system #1319

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

joshua-holmes
Copy link
Contributor

@joshua-holmes joshua-holmes commented Dec 7, 2024

  • Setup Wayland to use new mach object system
    • Store native state in new Native struct, removing all members from Wayland struct
    • In listeners, such as wl_registry_add_listener, rather than sharing a pointer to an instance of Linux with the Wayland server, pass the mach object system window id.
    • Store a pointer to Core in the global scope of Wayland so that listeners can use the provided window id and pointer to Core to lookup the data for that window, modify it, and save it directly in the handler function. See registryHandleGlobal() definition for an example.
    • Handle some errors that were previously unhandled
    • Handle getting and setting the state of windows when needed. Previously a step that wasn't done because we only had 1 state to manage, now we have 2: state in the local scope, and state in the mach object system (core.windows)
  • Setup logs to appropriately explain the current status of the X11 implementation, which is not functional

NOTE: Wayland runtime is still not working, but due to a sysgpu issue, it seems

  • By selecting this checkbox, I agree to license my contributions to this project under the license(s) described in the LICENSE file, and I have the right to do so or have received permission to do so by an employer or client I am producing work for whom has this right.

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.

1 participant