Stupid timeout bug fixed

[wxWidgets.git] / src / common / txtstrm.cpp

diff --git a/src/common/txtstrm.cpp b/src/common/txtstrm.cpp
index 3cb2c5bb5fe88f2a166c5940b9ee69ec27fa38c4..4e8dae20acbba014b49e15bfe75bfe1a7a87177c 100644 (file)
--- a/src/common/txtstrm.cpp
+++ b/src/common/txtstrm.cpp
@@ -38,8 +38,8 @@
 // wxTextInputStream
 // ----------------------------------------------------------------------------
 
-wxTextInputStream::wxTextInputStream(wxInputStream& s, const wxChar &sep)
-  : m_input(&s), m_string_separator(sep)
+wxTextInputStream::wxTextInputStream(wxInputStream &s, const wxString &sep)
+  : m_input(s), m_separators(sep)
 {
 }
 
@@ -47,58 +47,42 @@ wxTextInputStream::~wxTextInputStream()
 {
 }
 
-wxChar wxTextInputStream::NextNonWhiteSpace()
+wxChar wxTextInputStream::NextNonSeparators()
 {
     wxChar c = (wxChar) 0;
     for (;;)
     {
-        c = m_input->GetC();
        if (!m_input) return (wxChar) 0;
+        c = m_input.GetC();
        
-       if (c != wxT('\n') &&
+       if (c != wxT('\n') && 
            c != wxT('\r') &&
-           c != wxT('\t') &&
-           c != wxT(' '))
-       {
-           return c;
-       }
+           !m_separators.Contains(c)) 
+         return c;
     }
 
-    // this shouldn't happen
-    return (wxChar) 0;
 }
 
-void wxTextInputStream::SkipIfEndOfLine( wxChar c )
+inline bool wxTextInputStream::EatEOL(const wxChar &c)
 {
-    if (c == wxT('\n'))
+  if (c == wxT('\n')) return TRUE; // eat on UNIX
+  
+  if (c == wxT('\r')) // eat on both Mac and DOS
     {
-        // eat on UNIX
-       return;
+      if (!m_input) return TRUE;
+      wxChar c2 = m_input.GetC();
+      
+      if (c2 != wxT('\n'))  m_input.Ungetch( c2 ); // Don't eat on Mac
+      return TRUE;
     }
 
-    if (c == wxT('\r'))
-    {
-        // eat on both Mac and DOS
-       
-        wxChar c2 = m_input->GetC();
-        if (!m_input) return;
-       
-        if (c2 == wxT('\n'))
-        {
-           // eat on DOS
-           return;
-       }
-        else
-       {
-           // Don't eat on Mac
-            m_input->Ungetch( c2 );
-       }
-    }
-    else
-    {
-        // no line terminator
-        m_input->Ungetch( c );
-    }
+  return FALSE;
+}
+
+void wxTextInputStream::SkipIfEndOfLine( wxChar c )
+{
+    if (EatEOL(c)) return;
+    else m_input.Ungetch( c );  // no line terminator
 }
 
 wxUint32 wxTextInputStream::Read32()
@@ -107,25 +91,26 @@ wxUint32 wxTextInputStream::Read32()
     int sign;
     wxInt32 i;
 
-    int c = NextNonWhiteSpace();
     if (!m_input) return 0;
+    int c = NextNonSeparators(); 
+    if (c==(wxChar)0) return 0;
 
     i = 0;
     if (! (c == wxT('-') || c == wxT('+') || isdigit(c)) )
     {
-        m_input->Ungetch(c);
+        m_input.Ungetch(c);
         return 0;
     }
 
     if (c == wxT('-'))
     {
         sign = -1;
-        c = m_input->GetC();
+        c = m_input.GetC();
     } else
     if (c == wxT('+'))
     {
         sign = 1;
-        c = m_input->GetC();
+        c = m_input.GetC();
     } else
     {
         sign = 1;
@@ -134,7 +119,7 @@ wxUint32 wxTextInputStream::Read32()
     while (isdigit(c))
     {
         i = i*10 + (c - (int)wxT('0'));
-        c = m_input->GetC();
+        c = m_input.GetC();
     }
 
     SkipIfEndOfLine( c );
@@ -160,25 +145,26 @@ double wxTextInputStream::ReadDouble()
     double f;
     int sign;
 
-    int c = NextNonWhiteSpace();
-    if (!m_input) return 0.0;
+    if (!m_input) return 0;
+    int c = NextNonSeparators();
+    if (c==(wxChar)0) return 0;
 
     f = 0.0;
-    if (! (c == wxT('.') || c == wxT('-') || c == wxT('+') || isdigit(c)) )
+    if (! (c == wxT('.') || c == wxT(',') || c == wxT('-') || c == wxT('+') || isdigit(c)) )
     {
-        m_input->Ungetch(c);
+        m_input.Ungetch(c);
         return 0.0;
     }
 
     if (c == wxT('-'))
     {
         sign = -1;
-        c = m_input->GetC();
+        c = m_input.GetC();
     } else
     if (c == wxT('+'))
     {
         sign = 1;
-        c = m_input->GetC();
+        c = m_input.GetC();
     }
     else
     {
@@ -188,20 +174,20 @@ double wxTextInputStream::ReadDouble()
     while (isdigit(c))
     {
         f = f*10 + (c - wxT('0'));
-        c = m_input->GetC();
+        c = m_input.GetC();
     }
 
-    if (c == wxT('.'))
+    if (c == wxT('.') || c == wxT(','))
     {
         double f_multiplicator = (double) 0.1;
 
-        c = m_input->GetC();
+        c = m_input.GetC();
 
         while (isdigit(c))
        {
             f += (c-wxT('0'))*f_multiplicator;
             f_multiplicator /= 10;
-            c = m_input->GetC();
+            c = m_input.GetC();
         }
 
         if (c == wxT('e'))
@@ -209,7 +195,7 @@ double wxTextInputStream::ReadDouble()
             double f_multiplicator = 0.0;
             int i, e;
 
-            c = m_input->GetC();
+            c = m_input.GetC();
 
             switch (c)
            {
@@ -227,7 +213,7 @@ double wxTextInputStream::ReadDouble()
     }
     else
     {
-        m_input->Ungetch(c);
+        m_input.Ungetch(c);
     }
 
     f *= sign;
@@ -247,34 +233,10 @@ wxString wxTextInputStream::ReadLine()
 
     for (;;)
     {
-        c = m_input->GetC();
         if (!m_input) break;
+        c = m_input.GetC();
        
-        if (c == wxT('\n'))
-        {
-           // eat on UNIX
-           break;
-       }
-       
-        if (c == wxT('\r'))
-        {
-            // eat on both Mac and DOS
-       
-            wxChar c2 = m_input->GetC();
-           if (!m_input) break;
-       
-           if (c2 == wxT('\n'))
-            {
-               // eat on DOS
-               break;
-           }
-            else
-           {
-               // Don't eat on Mac
-                m_input->Ungetch( c2 );
-               break;
-           }
-        }
+       if (EatEOL(c)) break;
        
         line += c;
     }
@@ -284,47 +246,22 @@ wxString wxTextInputStream::ReadLine()
 
 wxString wxTextInputStream::ReadWord()
 {
-    wxChar c;
+    if (!m_input) return "";
+  
     wxString word;
-
-    if (m_string_separator==wxT(' ')) c=NextNonWhiteSpace();
-    else c = m_input->GetC();
+    wxChar c=NextNonSeparators();
+    if (c==(wxChar)0) return "";
 
     for (;;)
     {
-        if (!m_input) break;
+       if (m_separators.Contains(c)) break;
        
-        if (c == m_string_separator)
-         break;
-       
-        if (c == wxT('\n'))
-        {
-           // eat on UNIX
-           break;
-       }
-       
-        if (c == wxT('\r'))
-        {
-            // eat on both Mac and DOS
-       
-            wxChar c2 = m_input->GetC();
-           if (!m_input) break;
-       
-           if (c2 == wxT('\n'))
-            {
-               // eat on DOS
-               break;
-           }
-            else
-           {
-               // Don't eat on Mac
-                m_input->Ungetch( c2 );
-               break;
-           }
-        }
+       if (EatEOL(c)) break;
        
         word += c;
-        c = m_input->GetC();
+
+       if (!m_input) break;
+        c = m_input.GetC();
     }
 
     return word;
@@ -338,30 +275,15 @@ wxTextInputStream& wxTextInputStream::operator>>(wxString& word)
 
 wxTextInputStream& wxTextInputStream::operator>>(wxChar& c)
 {
-    wxChar c1 = m_input->GetC();
     if (!m_input)
     {
         c = (wxChar) 0;
         return *this;
     }
 
-    if (c1 == wxT('\r'))
-    {
-        c = wxT('\n');
-        wxChar c2 = m_input->GetC();
-       if (!m_input) return *this;
-       
-       if (c2 != wxT('\n'))
-       {
-           // we are on a Mac
-            m_input->Ungetch( c2 );
-       }
-    }
-    else
-    {
-        c = c1;
-    }
+    c = m_input.GetC();
 
+    if (EatEOL(c)) c=wxT('\n');
     return *this;
 }
 
@@ -401,15 +323,41 @@ wxTextInputStream& wxTextInputStream::operator>>(float& f)
     return *this;
 }
 
-wxTextOutputStream::wxTextOutputStream(wxOutputStream& s)
-  : m_output(&s)
+wxTextOutputStream::wxTextOutputStream(wxOutputStream& s, wxEOL mode)
+  : m_output(s)
 {
+    m_mode = mode;
+    if (m_mode == wxEOL_NATIVE)
+    {
+#if defined(__WXMSW__) || defined(__WXPM__)
+        m_mode = wxEOL_DOS;
+#elif defined(__WXMAC__)
+        m_mode = wxEOL_MAC;
+#else
+        m_mode = wxEOL_UNIX;
+#endif
+    }
 }
 
 wxTextOutputStream::~wxTextOutputStream()
 {
 }
 
+void wxTextOutputStream::SetMode( wxEOL mode = wxEOL_NATIVE )
+{
+    m_mode = mode;
+    if (m_mode == wxEOL_NATIVE)
+    {
+#if defined(__WXMSW__) || defined(__WXPM__)
+        m_mode = wxEOL_DOS;
+#elif defined(__WXMAC__)
+        m_mode = wxEOL_MAC;
+#else
+        m_mode = wxEOL_UNIX;
+#endif
+    }
+}
+
 void wxTextOutputStream::Write32(wxUint32 i)
 {
     wxString str;
@@ -449,29 +397,26 @@ void wxTextOutputStream::WriteString(const wxString& string)
         wxChar c = string[i];
         if (c == wxT('\n'))
         {
-#if   defined(__WINDOWS__)
-            c = wxT('\r');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-            c = wxT('\n');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-#elif defined(__UNIX__)
-            c = wxT('\n');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-#elif defined(__WXMAC__)
-            c = wxT('\r');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-#elif   defined(__OS2__)
-            c = wxT('\r');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-            c = wxT('\n');
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
-#else
-            #error  "wxTextOutputStream: unsupported platform."
-#endif
+           if (m_mode == wxEOL_DOS)
+           {
+                 c = wxT('\r');
+                 m_output.Write( (const void*)(&c), sizeof(wxChar) );
+                 c = wxT('\n');
+                 m_output.Write( (const void*)(&c), sizeof(wxChar) );
+           } else 
+           if (m_mode == wxEOL_MAC)
+           {
+                 c = wxT('\r');
+                 m_output.Write( (const void*)(&c), sizeof(wxChar) );
+           } else
+           {
+                 c = wxT('\n');
+                 m_output.Write( (const void*)(&c), sizeof(wxChar) );
+            }
         }
         else
         {
-            m_output->Write( (const void*)(&c), sizeof(wxChar) );
+            m_output.Write( (const void*)(&c), sizeof(wxChar) );
         }
    }
 }
@@ -496,25 +441,37 @@ wxTextOutputStream& wxTextOutputStream::operator<<(wxChar c)
 
 wxTextOutputStream& wxTextOutputStream::operator<<(wxInt16 c)
 {
-    Write16( (wxUint16)c );
+    wxString str;
+    str.Printf(wxT("%d"), (signed int)c);
+    WriteString(str);
+    
     return *this;
 }
 
 wxTextOutputStream& wxTextOutputStream::operator<<(wxInt32 c)
 {
-    Write32( (wxUint32)c );
+    wxString str;
+    str.Printf(wxT("%ld"), (signed long)c);
+    WriteString(str);
+    
     return *this;
 }
 
 wxTextOutputStream& wxTextOutputStream::operator<<(wxUint16 c)
 {
-    Write16(c);
+    wxString str;
+    str.Printf(wxT("%u"), (unsigned int)c);
+    WriteString(str);
+    
     return *this;
 }
 
 wxTextOutputStream& wxTextOutputStream::operator<<(wxUint32 c)
 {
-    Write32(c);
+    wxString str;
+    str.Printf(wxT("%lu"), (unsigned long)c);
+    WriteString(str);
+
     return *this;
 }