wxTextFile::Eof() bug corrected (always returned FALSE before)

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

diff --git a/src/common/filefn.cpp b/src/common/filefn.cpp
index c220105aba4885ae2099d7570ff2648fbd8096e8..59a9cd78f097a0bc4ef0072d88a8803cda6a735b 100644 (file)
--- a/src/common/filefn.cpp
+++ b/src/common/filefn.cpp
@@ -1451,6 +1451,19 @@ bool wxSetWorkingDirectory(const wxString& d)
 #endif
 }
 
+// Get the OS directory if appropriate (such as the Windows directory).
+// On non-Windows platform, probably just return the empty string.
+wxString wxGetOSDirectory()
+{
+#ifdef __WINDOWS__
+    char buf[256];
+    GetWindowsDirectory(buf, 256);
+    return wxString(buf);
+#else
+    return wxEmptyString;
+#endif
+}
+
 bool wxEndsWithPathSeparator(const char *pszFileName)
 {
   size_t len = Strlen(pszFileName);
@@ -1500,29 +1513,35 @@ void WXDLLEXPORT wxSplitPath(const char *pszFileName,
                              wxString *pstrName,
                              wxString *pstrExt)
 {
-  wxCHECK_RET( pszFileName, _("NULL file name in wxSplitPath") );
+  wxCHECK_RET( pszFileName, "NULL file name in wxSplitPath" );
 
   const char *pDot = strrchr(pszFileName, FILE_SEP_EXT);
   const char *pSepUnix = strrchr(pszFileName, FILE_SEP_PATH_UNIX);
   const char *pSepDos = strrchr(pszFileName, FILE_SEP_PATH_DOS);
 
-  // take the last of the two
+  // take the last of the two: nPosUnix containts the last slash in the
+  // filename
   size_t nPosUnix = pSepUnix ? pSepUnix - pszFileName : 0;
   size_t nPosDos = pSepDos ? pSepDos - pszFileName : 0;
   if ( nPosDos > nPosUnix )
     nPosUnix = nPosDos;
-//  size_t nLen = Strlen(pszFileName);
 
   if ( pstrPath )
     *pstrPath = wxString(pszFileName, nPosUnix);
-  if ( pDot ) {
-    size_t nPosDot = pDot - pszFileName;
+
+  size_t nPosDot = 0;
+  if ( pDot )
+    nPosDot = pDot - pszFileName;
+
+  if ( nPosDot > nPosUnix ) {
+    // the file name looks like "path/name.ext"
     if ( pstrName )
       *pstrName = wxString(pszFileName + nPosUnix + 1, nPosDot - nPosUnix);
     if ( pstrExt )
       *pstrExt = wxString(pszFileName + nPosDot + 1);
   }
   else {
+    // there is either no dot at all or there is a '/' after it
     if ( pstrName )
       *pstrName = wxString(pszFileName + nPosUnix + 1);
     if ( pstrExt )