Fixed some issues from PR review

Author: ed7coyne

examples/FirebaseSerialHost_ESP8266/FirebaseSerialHost_ESP8266.ino

@@ -15,8 +15,8 @@
 //
 
 
-// FirebasePush_ESP8266 is a sample that will start our serial transciever
-// listening on a software port and allow debug over the main serial port.
+// A sample that will start our serial transciever listening on a software
+// port and allow debug over the main serial port.
 //
 // A suggested setup for testing this example would be a USB to TTL cable
 // with the green wire connected to pin 5 and the white wire connected to

src/SerialTransceiver.h

@@ -1,28 +1,3 @@
-#ifndef MODEM_SERIAL_TRANSCIEVER_H
-#define MODEM_SERIAL_TRANSCIEVER_H
-
-#include <memory>
-
-#include "Firebase.h"
-#include "modem/commands.h"
-
-namespace firebase {
-namespace modem {
-
-class SerialTransceiver {
- public:
-  void begin(Stream* serial);
-  void loop();
-
- private:
-  std::unique_ptr<Command> CreateCommand(const String& name, Firebase* fbase);
-
-  std::unique_ptr<Firebase> fbase_;
-  std::unique_ptr<ArduinoInputStream> in_;
-  std::unique_ptr<ArduinoOutputStream> out_;
-};
-
-}  // modem
-}  // firebase
-
-#endif // MODEM_SERIAL_TRANSCIEVER_H
+#include "modem/SerialTransceiver.h"
+// Bring them into the base namespace for easier use in arduino ide.
+using firebase::modem::SerialTransceiver;

src/SerialTransceiver.cpp renamed to src/modem/SerialTransceiver.cpp

@@ -0,0 +1,28 @@
+#ifndef MODEM_SERIAL_TRANSCIEVER_H
+#define MODEM_SERIAL_TRANSCIEVER_H
+
+#include <memory>
+
+#include "Firebase.h"
+#include "modem/commands.h"
+
+namespace firebase {
+namespace modem {
+
+class SerialTransceiver {
+ public:
+  void begin(Stream* serial);
+  void loop();
+
+ private:
+  std::unique_ptr<Command> CreateCommand(const String& name, Firebase* fbase);
+
+  std::unique_ptr<Firebase> fbase_;
+  std::unique_ptr<ArduinoInputStream> in_;
+  std::unique_ptr<ArduinoOutputStream> out_;
+};
+
+}  // modem
+}  // firebase
+
+#endif // MODEM_SERIAL_TRANSCIEVER_H