SF_printable_fix #130

Merged
drmpf merged 1 commits from SF_fixes into master 2021-05-13 10:06:16 +03:00
drmpf commented 2021-05-13 06:57:24 +03:00 (Migrated from github.com)

User requested SafeString library support be added for this project.
https://forum.arduino.cc/t/libraries-updated-to-support-new-rp2040-based-boards-raspberry-pi-pico-etc/860353/3
Current SafeString library works for Arduino Mbed OS RP2040 V2.0.0 board install

This pull request fixes missing Printable.h file
But in order to complete adding the support need a unique define (-D... = .. ) that can be detected by SafeString.h so that the
necessary
namespace arduino {
can be set
Arduino Mbed OS RP2040 V2.0.0 has a -DARDUINO_ARCH_MBED_RP2040 define

Can you add a -D to your compile line (and when Arduino detects libraries)
Also dtostrf( ) declaration not found by SafeString library (works with Arduino pico, fixed by an extern ... in SafeString.h )

User requested SafeString library support be added for this project. https://forum.arduino.cc/t/libraries-updated-to-support-new-rp2040-based-boards-raspberry-pi-pico-etc/860353/3 Current SafeString library works for Arduino Mbed OS RP2040 V2.0.0 board install This pull request fixes missing Printable.h file But in order to complete adding the support need a unique define (-D... = .. ) that can be detected by SafeString.h so that the necessary namespace arduino { can be set Arduino Mbed OS RP2040 V2.0.0 has a -DARDUINO_ARCH_MBED_RP2040 define Can you add a -D to your compile line (and when Arduino detects libraries) Also dtostrf( ) declaration not found by SafeString library (works with Arduino pico, fixed by an extern ... in SafeString.h )
earlephilhower commented 2021-05-13 10:06:21 +03:00 (Migrated from github.com)

I've submitted the -D additions with
https://github.com/PowerBroker2/SafeString/pull/35

I've submitted the -D additions with https://github.com/PowerBroker2/SafeString/pull/35
earlephilhower commented 2021-05-13 10:16:30 +03:00 (Migrated from github.com)

#131 adds the non ISO stuff to standard Arduino includes, so covers dtostrf, even though I don't see it used...

#131 adds the non ISO stuff to standard Arduino includes, so covers dtostrf, even though I don't see it used...
Sign in to join this conversation.