[gnoduino: 147/237] Fixing warnings in Stream (Paul Stoffregen)

[Lucian Langa <lucilanga src gnome org>]

commit 6b94385966426e0be5785ea476694d96946461e4
Author: David A. Mellis <d mellis arduino cc>
Date:   Sun Oct 2 09:44:21 2011 -0400

    Fixing warnings in Stream (Paul Stoffregen)
    
    http://code.google.com/p/arduino/issues/detail?id=208

 arduino/cores/arduino/Stream.cpp |    9 ++++-----
 arduino/cores/arduino/Stream.h   |    6 +++---
 2 files changed, 7 insertions(+), 8 deletions(-)
---
diff --git a/arduino/cores/arduino/Stream.cpp b/arduino/cores/arduino/Stream.cpp
index 8b6f1e8..a7d9d38 100644
--- a/arduino/cores/arduino/Stream.cpp
+++ b/arduino/cores/arduino/Stream.cpp
@@ -67,9 +67,9 @@ int Stream::peekNextDigit()
 // Public Methods
 //////////////////////////////////////////////////////////////
 
-void Stream::setTimeout( long timeout)  // sets the maximum number of milliseconds to wait
+void Stream::setTimeout(unsigned long timeout)  // sets the maximum number of milliseconds to wait
 {
-   this->_timeout = timeout;
+  _timeout = timeout;
 }
 
  // find returns true if the target string is found
@@ -165,7 +165,7 @@ long Stream::parseInt(char skipChar)
 // as parseInt but returns a floating point value
 float Stream::parseFloat()
 {
-  parseFloat(NO_SKIP_CHAR);
+  return parseFloat(NO_SKIP_CHAR);
 }
 
 // as above but the given skipChar is ignored
@@ -174,7 +174,6 @@ float Stream::parseFloat(char skipChar){
   boolean isNegative = false;
   boolean isFraction = false;
   long value = 0;
-  float fValue;
   char c;
   float fraction = 1.0;
 
@@ -223,7 +222,7 @@ int Stream::readBytes( char *buffer, size_t length)
 
 int Stream::readBytesUntil( char terminator, char *buffer, size_t length)
 {
- int index = 0;
+    unsigned int index = 0;
     *buffer = 0;
     while(index < length-1 ){
       int c = timedRead();
diff --git a/arduino/cores/arduino/Stream.h b/arduino/cores/arduino/Stream.h
index 3c6d025..8f2de1e 100644
--- a/arduino/cores/arduino/Stream.h
+++ b/arduino/cores/arduino/Stream.h
@@ -38,8 +38,8 @@ readBytesBetween( pre_string, terminator, buffer, length)
 class Stream : public Print
 {
   private:
-    long _timeout;      // number of milliseconds to wait for the next char before aborting timed read
-    long _startMillis;  // used for timeout measurement
+    unsigned long _timeout;      // number of milliseconds to wait for the next char before aborting timed read
+    unsigned long _startMillis;  // used for timeout measurement
     int timedRead();    // private method to read stream with timeout
     int timedPeek();    // private method to peek stream with timeout
     int peekNextDigit(); // returns the next numeric digit in the stream or -1 if timeout
@@ -54,7 +54,7 @@ class Stream : public Print
 
 // parsing methods
 
-  void setTimeout(long timeout);  // sets maximum milliseconds to wait for stream data, default is 1 second
+  void setTimeout(unsigned long timeout);  // sets maximum milliseconds to wait for stream data, default is 1 second
 
   bool find(char *target);   // reads data from the stream until the target string is found
   // returns true if target string is found, false if timed out (see setTimeout)