Changing String::toCharArray() and getBytes() to accept a buffer, rather than return one. That way they don't expose the internal representation of the String class, allowing future optimization. Thanks to Paul Stoffregen.

1 files changed, 3 insertions, 1 deletions

diff --git a/cores/arduino/Print.cpp b/cores/arduino/Print.cpp
index 6fe162d..4ee556d 100755
--- a/cores/arduino/Print.cpp
+++ b/cores/arduino/Print.cpp
@@ -45,7 +45,9 @@ void Print::write(const uint8_t *buffer, size_t size)
 
 void Print::print(const String &s)
 {
-  print(s.toCharArray());
+  for (int i = 0; i < s.length(); i++) {
+    write(s[i]);
+  }
 }
 
 void Print::print(const char str[])