-
Notifications
You must be signed in to change notification settings - Fork 72
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
Window Manager not working #151
Comments
Is this using unmodified wm example? |
yes |
I've found out that |
can you give more details on this? |
It only calls the callback WHEN an error occurs, I've never been able to have a callback execute when there is no error. |
afaik this is generic problem and I don't know good solution Because of the way protocol is optimised for latency you generate id on the client, then submit request using this id most requests don't need to return anything and only return errors. There is no way to tell if /when request completed |
The only thing protocol guarantees is if you have request A and then request B they performed in order on server |
Probably callback style is bad api for this, maybe we should return EventEmitter from every request Most requests never return anything and only get response when there is error ( CreateWindow, ChangeWindowAttributes ) Some always return data or error exactly once ( GetWindowAttributes, GetAtomName, GetProperty) Some call callback one or more times ( ListFontsWithInfo: This request is similar to ListFonts, but it also returns information about |
An event emitter would make more sense an would be easier to implement. |
I get this error from the window manager example:
The text was updated successfully, but these errors were encountered: