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.

Sign up for GitHub

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

Jump to bottom

fix more array.tostring() removal issues in Python 3.9 #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix more array.tostring() removal issues in Python 3.9 #77

wants to merge 1 commit into from

Commits on Oct 17, 2022

  1. fix more array.tostring() removal issues in Python 3.9 

    Several shell.py script versions use the depreciated array.tostring() method,
which has been removed in Python 3.9. I changed these instances to .tobytes(),
and adjusted .ljust()'s second argument to be a byte string.

I also fixed a related issue, where #flash'ing from a shell.py which assumes a
2-cell wide Forth to a non-.hex file would error
because the unsigned short array returned from serialize() would be written to
file with assumptions of signed short:

open(dest, "wb").write(array.array("h", d).tobytes())

This would cause an overflow error. Changing "h" to "H" in that last code
snippet (in swapforth.py), lines up the signdedness.
    @stuij
    stuij committed Oct 17, 2022
    Configuration menu
    Copy the full SHA
    46c61a9 View commit details
    Browse the repository at this point in the history