Changeset 1117

Timestamp:

Mar 5, 2013, 11:30:56 PM (12 years ago)

Author:

Dmitry A. Kuminov

Message:

corelib: Make QFileInfo returned by QDirIterator report symlinks properly.

In QT_OS2_USE_DOSFINDFIRST mode (which is ON now), we use
DosFindFirst which is unaware of symlinks and doesn't properly resolve that.
Given that in many other places kLIBC functions are used which handle symlinks
pretty well, this would completely confuse some apps like QFileDialog (see #284).

This fix removes this inconsistency.

Location:

trunk/src/corelib/io

Files:

• qfsfileengine_iterator_os2.cpp (modified) (2 diffs)
• qfsfileengine_os2.cpp (modified) (2 diffs)

trunk/src/corelib/io/qfsfileengine_iterator_os2.cpp

@@ -52 +52 @@
 #include <QtCore/qvariant.h>
 
+
 #ifndef QT_OS2_USE_DOSFINDFIRST
 #include <dirent.h>
-#include <unistd.h>
 #else
 #include <QtCore/qdatetime.h>
@@ -360 +360 @@
     }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     if (ffb->attrFile & FILE_DIRECTORY)
         d_ptr->fileFlags |= QAbstractFileEngine::DirectoryType;

trunk/src/corelib/io/qfsfileengine_os2.cpp

@@ -42 +42 @@
 ****************************************************************************/
 
-// temporary, until struct dirent in kLIBC gets creation and access time fields
-#define QT_OS2_USE_DOSFINDFIRST
-
 #include "qplatformdefs.h"
 #include "qabstractfileengine.h"
@@ -677 +674 @@
             is_link = false;        // drive/share names are never symlinks
         } else {
-#ifdef QT_OS2_USE_DOSFINDFIRST
-            is_link = false;
-#else
             QT_STATBUF st;          // don't clobber our main one
             that->is_link = (QT_LSTAT(nativeFilePath.constData(), &st) == 0) ?
                             S_ISLNK(st.st_mode) : false;
-#endif
         }
     }