Fixs from testing streaming example

ed7coyne · ed7coyne · commit 326518c1744b · 2016-05-26T16:32:15.000-07:00
diff --git a/examples/FirebaseSerialHost_ESP8266/push.txt b/examples/FirebaseSerialHost_ESP8266/push.txt
@@ -1 +1 @@
-PUSH /seria/push_test "this is a test string \ "
+PUSH /serial/push_test "this is a test string \ "
diff --git a/src/SerialTransceiver.h b/src/SerialTransceiver.h
@@ -1,3 +1,7 @@
+#include <string>
+// This is needed to compile std::string on esp8266.
+template class std::basic_string<char>;
+
 #include "modem/SerialTransceiver.h"
 // Bring them into the base namespace for easier use in arduino ide.
 using firebase::modem::SerialTransceiver;
diff --git a/src/modem/json_util.h b/src/modem/json_util.h
@@ -16,8 +16,8 @@ std::string ReplaceAll(std::string str, const std::string& from, const std::stri
 
 // TODO(edcoyne): We should use a json library to escape.
 inline std::string EncodeForJson(std::string input) {
-  ReplaceAll(input, "\\", "\\\\");
-  ReplaceAll(input, "\"", "\\\"");
+  input = ReplaceAll(input, "\\", "\\\\");
+  input = ReplaceAll(input, "\"", "\\\"");
   return "\"" + input + "\"";
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-PUSH /seria/push_test "this is a test string \ "`
	`1`	`+PUSH /serial/push_test "this is a test string \ "`
Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ std::string ReplaceAll(std::string str, const std::string& from, const std::stri`
`16`	`16`
`17`	`17`	`// TODO(edcoyne): We should use a json library to escape.`
`18`	`18`	`inline std::string EncodeForJson(std::string input) {`
`19`		`- ReplaceAll(input, "\\", "\\\\");`
`20`		`- ReplaceAll(input, "\"", "\\\"");`
	`19`	`+ input = ReplaceAll(input, "\\", "\\\\");`
	`20`	`+ input = ReplaceAll(input, "\"", "\\\"");`
`21`	`21`	`return "\"" + input + "\"";`
`22`	`22`	`}`
`23`	`23`