Skip to content

Use bytearray instead of BytesIO in MspDecoder - #9877

Open
radarhere wants to merge 2 commits into
python-pillow:mainfrom
radarhere:msp_variable
Open

Use bytearray instead of BytesIO in MspDecoder#9877
radarhere wants to merge 2 commits into
python-pillow:mainfrom
radarhere:msp_variable

Conversation

@radarhere

Copy link
Copy Markdown
Member
  1. When iterating through rows, the row index is assigned to x

    for x, rowlen in enumerate(rowmap):

    y would seem to be the correct choice instead.

  2. MspDecoder writes to a io.BytesIO instance, only to retrieve the value from it once done. Using bytearray() would be simpler.

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