From 8674a1b9e9da542b729d660d8f996f0e58008dbd Mon Sep 17 00:00:00 2001
From: Cristian Maglie <c.maglie@bug.st>
Date: Wed, 12 Jun 2013 13:20:25 +0200
Subject: [PATCH] Refactored String methods and constructor on Process class.

---
 libraries/Bridge/Process.cpp | 14 ++------------
 libraries/Bridge/Process.h   | 11 ++++-------
 2 files changed, 6 insertions(+), 19 deletions(-)

diff --git a/libraries/Bridge/Process.cpp b/libraries/Bridge/Process.cpp
index 6b86d54..78f168f 100644
--- a/libraries/Bridge/Process.cpp
+++ b/libraries/Bridge/Process.cpp
@@ -66,22 +66,12 @@ void Process::doBuffer() {
   buffered = bridge.transfer(cmd, 3, buffer, sizeof(buffer));
 }
 
-void Process::begin(String &command) {
+void Process::begin(const String &command) {
   close();
   cmdline = new String(command);
 }
 
-void Process::begin(const char *command) {
-  close();
-  cmdline = new String(command);
-}
-
-void Process::addParameter(const char *param) {
-  *cmdline += "\xFE";
-  *cmdline += param;
-}
-
-void Process::addParameter(String &param) {
+void Process::addParameter(const String &param) {
   *cmdline += "\xFE";
   *cmdline += param;
 }
diff --git a/libraries/Bridge/Process.h b/libraries/Bridge/Process.h
index 2877545..cf950d4 100644
--- a/libraries/Bridge/Process.h
+++ b/libraries/Bridge/Process.h
@@ -23,16 +23,13 @@
 
 class Process : public Stream {
 public:
-  // Default constructor uses global Bridge instance
-  Process() : bridge(Bridge), started(false), buffered(0), readPos(0) { }
   // Constructor with a user provided BridgeClass instance
-  Process(BridgeClass &_b) : bridge(_b), started(false), buffered(0), readPos(0) { }
+  Process(BridgeClass &_b = Bridge) :
+	  bridge(_b), started(false), buffered(0), readPos(0) { }
   ~Process();
   
-  void begin(String &command);
-  void begin(const char *command);
-  void addParameter(String &param);
-  void addParameter(const char *param);
+  void begin(const String &command);
+  void addParameter(const String &param);
   unsigned int run();
   void runAsynchronously();
   boolean running();