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

Update RF24_config.h #502

Merged
merged 4 commits into from
Jun 18, 2019
Merged

Update RF24_config.h #502

merged 4 commits into from
Jun 18, 2019

Conversation

IkpeohaGodson
Copy link
Contributor

I added the following after line 135 in order for RF24 library to compile on ESP32 without error:
#elif define (ESP32)
#include <pgmspace.h>
#define PRIPSTR "%s"

@Avamander
Copy link
Member

Does PRIPSTR really still not exist on latest ESP32 core?

Avamander
Avamander previously approved these changes Jun 18, 2019
Copy link
Member

@Avamander Avamander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cleaned it up and it should be fine to merge now

Updated copyright header
Avamander
Avamander previously approved these changes Jun 18, 2019
Copy link
Member

@Avamander Avamander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed copyright header as well

Fixed another inconsistency
@Avamander Avamander merged commit 5bc1245 into nRF24:master Jun 18, 2019
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