speed optimizations: some functions now use wxString::Alloc, wxTextFile::Read

uses a buffer to read data (instead of reading it one char at a time)


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@276 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/common/config.cpp

@@ -128,6 +128,7 @@ wxConfig::PathChanger::~PathChanger()
 wxString ExpandEnvVars(const wxString& str)
 {
   wxString strResult;
+  strResult.Alloc(str.Len());
 
   // don't change the values the enum elements: they must be equal
   // to the matching [closing] delimiter.

src/common/fileconf.cpp

@@ -221,8 +221,8 @@ void wxFileConfig::Parse(wxTextFile& file, bool bLocal)
       }
 
       if ( *pEnd != ']' ) {
-        wxLogError("file '%s': unexpected character at line %d (missing ']'?)",
-                   file.GetName(), n + 1);
+        wxLogError("file '%s': unexpected character %c at line %d.",
+                   file.GetName(), *pEnd, n + 1);
         continue; // skip this line
       }
 
@@ -950,6 +950,7 @@ void wxFileConfig::ConfigEntry::SetDirty()
 wxString FilterIn(const wxString& str)
 {
   wxString strResult;
+  strResult.Alloc(str.Len());
 
   bool bQuoted = !str.IsEmpty() && str[0] == '"';
 
@@ -989,6 +990,7 @@ wxString FilterIn(const wxString& str)
 wxString FilterOut(const wxString& str)
 {
   wxString strResult;
+  strResult.Alloc(str.Len());
 
   // quoting is necessary to preserve spaces in the beginning of the string
   bool bQuote = isspace(str[0]) || str[0] == '"';

src/common/textfile.cpp

@@ -115,6 +115,7 @@ wxTextFile::Type wxTextFile::GuessType() const
       case Type_Unix: nUnix++; break;   \
       case Type_Dos:  nDos++;  break;   \
       case Type_Mac:  nMac++;  break;   \
+      default: wxFAIL_MSG("unknown line terminator"); \
     }
 
   uint n;
@@ -159,10 +160,12 @@ bool wxTextFile::Read()
 
   wxString str;
   char ch, chLast = '\0';
+  char buf[1024];
+  int n, nRead;
   while ( !m_file.Eof() ) {
-    // @@ should really use a buffer for efficiency
-    if ( m_file.Read(&ch, sizeof(ch)) == ofsInvalid ) {
-      // read error
+    nRead = m_file.Read(buf, WXSIZEOF(buf));
+    if ( nRead == ofsInvalid ) {
+      // read error (error message already given in wxFile::Read)
       m_file.Close();
       return FALSE;
     }
@@ -171,6 +174,8 @@ bool wxTextFile::Read()
       #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
@@ -203,6 +208,7 @@ bool wxTextFile::Read()
           }
       }
     }
+  }
 
   // anything in the last line?
   if ( !str.IsEmpty() ) {