wxWindow::Fit() infinite loop bug fixed

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

diff --git a/src/common/textfile.cpp b/src/common/textfile.cpp
index cf3095cd5e18b6ad32dfaea71d9f0c401868a9a2..c70d51d278f3e2db6683ccd23bf7b9b208311a1d 100644 (file)
--- a/src/common/textfile.cpp
+++ b/src/common/textfile.cpp
@@ -38,17 +38,15 @@
 // ----------------------------------------------------------------------------
 
 // default type is the native one
-const wxTextFile::Type wxTextFile::typeDefault = wxTextFile::
+const wxTextFileType wxTextFile::typeDefault =
 #if   defined(__WINDOWS__)
-  Type_Dos;
+  wxTextFileType_Dos;
 #elif defined(__UNIX__)
-  Type_Unix;
+  wxTextFileType_Unix;
 #elif defined(__WXMAC__)
-  Type_Mac;
-  // if you feel brave, remove the next line
-  #error  "wxTextFile: code for Mac files is untested."
+  wxTextFileType_Mac;
 #else
-  Type_None;
+  wxTextFileType_None;
   #error  "wxTextFile: unsupported platform."
 #endif
 
@@ -73,6 +71,7 @@ wxTextFile::~wxTextFile()
 bool wxTextFile::Open(const wxString& strFile)
 {
   m_strFile = strFile;
+
   return Open();
 }
 
@@ -95,7 +94,7 @@ bool wxTextFile::Open()
 
 // analyse some lines of the file trying to guess it's type.
 // if it fails, it assumes the native type for our platform.
-wxTextFile::Type wxTextFile::GuessType() const
+wxTextFileType wxTextFile::GuessType() const
 {
   // file should be opened and we must be in it's beginning
   wxASSERT( m_file.IsOpened() && m_file.Tell() == 0 );
@@ -112,9 +111,9 @@ wxTextFile::Type wxTextFile::GuessType() const
 
   #define   AnalyseLine(n)              \
     switch ( m_aTypes[n] ) {            \
-      case Type_Unix: nUnix++; break;   \
-      case Type_Dos:  nDos++;  break;   \
-      case Type_Mac:  nMac++;  break;   \
+      case wxTextFileType_Unix: nUnix++; break;   \
+      case wxTextFileType_Dos:  nDos++;  break;   \
+      case wxTextFileType_Mac:  nMac++;  break;   \
       default: wxFAIL_MSG(_("unknown line terminator")); \
     }
 
@@ -135,17 +134,21 @@ wxTextFile::Type wxTextFile::GuessType() const
   }
   else {
     #define   GREATER_OF(t1, t2) n##t1 == n##t2 ? typeDefault               \
-                                                : n##t1 > n##t2 ? Type_##t1 \
-                                                                : Type_##t2
+                                                : n##t1 > n##t2             \
+                                                    ? wxTextFileType_##t1   \
+                                                    : wxTextFileType_##t2
 
+    // Watcom C++ doesn't seem to be able to handle the macro
+#if !defined(__WATCOMC__)
     if ( nDos > nUnix )
       return GREATER_OF(Dos, Mac);
     else if ( nDos < nUnix )
       return GREATER_OF(Unix, Mac);
     else {
       // nDos == nUnix
-      return nMac > nDos ? Type_Mac : typeDefault;
+      return nMac > nDos ? wxTextFileType_Mac : typeDefault;
     }
+#endif // __WATCOMC__
 
     #undef    GREATER_OF
   }
@@ -170,17 +173,14 @@ bool wxTextFile::Read()
       return FALSE;
     }
 
-    #ifdef  __MAC__
-      #pragma message("wxTextFile::Read() hasn't been tested with Mac files.")
-    #endif
-
     for ( n = 0; n < nRead; n++ ) {
       ch = buf[n];
       switch ( ch ) {
         case '\n':
           // Dos/Unix line termination
           m_aLines.Add(str);
-          m_aTypes.Add(chLast == '\r' ? Type_Dos : Type_Unix);
+          m_aTypes.Add(chLast == '\r' ? wxTextFileType_Dos
+                                      : wxTextFileType_Unix);
           str.Empty();
           chLast = '\n';
           break;
@@ -189,7 +189,7 @@ bool wxTextFile::Read()
           if ( chLast == '\r' ) {
             // Mac empty line
             m_aLines.Add("");
-            m_aTypes.Add(Type_Mac);
+            m_aTypes.Add(wxTextFileType_Mac);
           }
           else
             chLast = '\r';
@@ -199,7 +199,8 @@ bool wxTextFile::Read()
           if ( chLast == '\r' ) {
             // Mac line termination
             m_aLines.Add(str);
-            m_aTypes.Add(Type_Mac);
+            m_aTypes.Add(wxTextFileType_Mac);
+            chLast = ch;
             str = ch;
           }
           else {
@@ -212,14 +213,23 @@ bool wxTextFile::Read()
 
   // anything in the last line?
   if ( !str.IsEmpty() ) {
-    m_aTypes.Add(Type_None);  // no line terminator
+    m_aTypes.Add(wxTextFileType_None);  // no line terminator
     m_aLines.Add(str);
   }
 
   return TRUE;
 }
 
-bool wxTextFile::Write(Type typeNew)
+bool wxTextFile::Close()
+{
+    m_aTypes.Clear();
+    m_aLines.Clear();
+    m_nCurLine = 0;
+
+    return TRUE;
+}
+
+bool wxTextFile::Write(wxTextFileType typeNew)
 {
   wxTempFile fileTmp(m_strFile);
 
@@ -231,9 +241,25 @@ bool wxTextFile::Write(Type typeNew)
   size_t nCount = m_aLines.Count();
   for ( size_t n = 0; n < nCount; n++ ) {
     fileTmp.Write(m_aLines[n] +
-                  GetEOL(typeNew == Type_None ? m_aTypes[n] : typeNew));
+                  GetEOL(typeNew == wxTextFileType_None ? m_aTypes[n]
+                                                        : typeNew));
   }
 
   // replace the old file with this one
   return fileTmp.Commit();
 }
+
+const wxChar *wxTextFile::GetEOL(wxTextFileType type)
+  {
+    switch ( type ) {
+      case wxTextFileType_None: return _T("");
+      case wxTextFileType_Unix: return _T("\n");
+      case wxTextFileType_Dos:  return _T("\r\n");
+      case wxTextFileType_Mac:  return _T("\r");
+
+      default:
+        wxFAIL_MSG(_T("bad file type in wxTextFile::GetEOL."));
+        return (const wxChar *) NULL;
+    }
+  }
+