X Tutup
The Wayback Machine - https://web.archive.org/web/20210106130902/https://github.com/python-xlib/python-xlib/pull/187
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

[fix bug]-cannot use a str to initialize an array with typecode 'B' #187

Open
wants to merge 1 commit into
base: master
from

Conversation

@jinxiangwzh
Copy link

@jinxiangwzh jinxiangwzh commented Dec 21, 2020

No description provided.

@petli
Copy link
Contributor

@petli petli commented Dec 21, 2020

Thank you for the PR, but this change breaks a lot of unit tests (see the CI build failure), so can you look at that and see what you need change in the code to ensure they still work, please?

I'm also not sure what the bug is here. Could you explain to which methods or requests you want to pass in a string to this structure, rather than controlling the encoding of it to an array yourself? I'm thinking that such encoding could make more sense in the wrapping methods in the X objects (Display, Window etc), rather than this low down in the protocol implementation.

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

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.
X Tutup