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

Add string constructor and concat routines taking explicit length args #5586

Merged
merged 5 commits into from
Aug 31, 2021

Conversation

rcombs
Copy link
Contributor

@rcombs rcombs commented Aug 24, 2021

Summary

Applies the upstream changes here: arduino/ArduinoCore-API@3b88acac8^...0d83f1a

Impact

Adds new String convenience methods that are now available in the mainline Arduino implementation, simplifying interoperability with C code that uses pointer+length strings rather than 0-termination. Also includes a change to avoid mutating the source string when taking a substring.

rcombs added 2 commits August 24, 2021 03:17
From upstream ArduinoCore-API commit 3b88acac8ee00fc57bc3328e68823aa53088f891
From upstream ArduinoCore-API commit 84314888a8a73b643469c5609beed6d21d1c037a
@CLAassistant
Copy link

CLAassistant commented Aug 24, 2021

CLA assistant check
All committers have signed the CLA.

rcombs added 3 commits August 25, 2021 14:16
From upstream ArduinoCore-API commit dd236bfd2c62ed083aaa1b1229d71e85fd92f7a9
From upstream ArduinoCore-API commit eaab14db09073afc20bcf26bd9b7448a6a1309b0
From upstream ArduinoCore-API commit 0d83f1afc3367037dbde5323c2abd0ae1bd2c583
@rcombs rcombs force-pushed the string-explicit-length branch from c11e99d to 2000160 Compare August 25, 2021 19:16
@me-no-dev me-no-dev merged commit 24b76cb into espressif:master Aug 31, 2021
@me-no-dev
Copy link
Member

thanks @rcombs :)

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.

3 participants