-
Notifications
You must be signed in to change notification settings - Fork 1
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
Backend/x11 #22
Backend/x11 #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments explaining the convertion process would be nice ^^
Maybe with doc links
Regarding your async mention: We should provide an uniform interface. For example if implementing dbus we will also need to wait for some sort of callback. |
hm... isn't the code not readable enough? Aren't the variable names + function names explaining enough? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has to be adapted to our OutputInfo interface
30149da
to
c16d527
Compare
Adding the backend implementation for
X11
.Closes #3
General information
I'm aware, that you need
async
for wayland. But that's not needed forxorg
and we can still call this function in an async function.