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

ports/esp32/mpconfigport.h: Enable reverse/inplace special methods. #7613

Closed
wants to merge 1 commit into from

Conversation

jimmo
Copy link
Member

@jimmo jimmo commented Aug 4, 2021

See #5897 and https://forum.micropython.org/viewtopic.php?f=3&t=10927&p=60120

Reverse ops are supported on STM32, rp2, etc. Anything that's enabled for STM32 probably should also be on ESP32.

Also added inplace arithmetic. This will be the first port to enable this, but we have the flash space on ESP32.

Signed-off-by: Jim Mussared <jim.mussared@gmail.com>
@dpgeorge
Copy link
Member

dpgeorge commented Aug 4, 2021

Reverse ops are supported on STM32, rp2, etc. Anything that's enabled for STM32 probably should also be on ESP32.

+1

Also added inplace arithmetic. This will be the first port to enable this, but we have the flash space on ESP32.

Probably also a good idea, but then rp2 and stm32 would want to follow suit (and for small stm32 builds we'd want to have the option in the board config to turn it off).

@dpgeorge
Copy link
Member

dpgeorge commented Aug 7, 2021

Merged in bfc8e88

@dpgeorge dpgeorge closed this Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants