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
bytearray.remove cannot remove bytes with value greater than 127 #71226
Comments
The following code fails with a ValueError (but I expect it to succeed):
Tested with Python 2.7.11 and 3.5.1. Probably it's a case of comparing a char (signed byte) with an unsigned value. |
Test is not catching it, because it only deals with ascii part. |
This seems to just be a bug in the implementation of remove. I have a patch to fix this and a test case. |
LGTM. |
But we first need to test that value is in the range from 0 to 255. |
1 similar comment
But we first need to test that value is in the range from 0 to 255. |
Ah, this is already checked in the converter. |
New changeset 10444778d41c by Serhiy Storchaka in branch '3.5': New changeset 9acf44b7ff7b by Serhiy Storchaka in branch '2.7': New changeset 3eb502d43616 by Serhiy Storchaka in branch 'default': |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: