Skip to content

Fix pgm_read_ptr() #5735

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

Merged
merged 1 commit into from
Feb 7, 2019
Merged

Conversation

earlephilhower
Copy link
Collaborator

Fix dereferencing in pgm_read_ptr() macro on 8266 and host.

Fixes #5733

Fix dereferencing in pgm_read_ptr() macro on 8266 and host.

Fixes esp8266#5733
earlephilhower added a commit to earlephilhower/Arduino that referenced this pull request Feb 7, 2019
@earlephilhower earlephilhower merged commit 29bb7fc into esp8266:master Feb 7, 2019
@earlephilhower earlephilhower deleted the morepgmptrfix branch February 7, 2019 18:47
@Lauszus
Copy link
Contributor

Lauszus commented Apr 5, 2019

When is the next released planned? As it was also noted in #5733 this issue breaks compatibility with the USB host library and causes our Travis build to fail: https://travis-ci.org/felis/USB_Host_Shield_2.0/jobs/516275113.

earlephilhower added a commit that referenced this pull request Apr 11, 2019
* Unaligned access support for pgm_read_word/dword

* Fix pgm_read_ptr_aligned() per #5735

* Allow users to use aligned-only via a #define

Adding -DPGM_READ_UNALIGNED=0 or #define PGM_READ_UNALIGNED 0 will
change the default at compile-time to only aligned (faster, but less
compatible) macro implementations.

Default is still to allow unaligned accesses.
Lauszus added a commit to felis/USB_Host_Shield_2.0 that referenced this pull request Apr 14, 2019
tomec-martin added a commit to tomec-martin/ArduinoFake that referenced this pull request Aug 8, 2021
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.

2 participants