Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-25088

Qt 5 to-do's in QFileSystemModel

XMLWordPrintable

    • 94519a441cf1ea77f1422c44a7ef8ec15171ad04

      The QFileSystemModel code contains three Qt 5 to-do comments:

      qtbase/src/widgets/dialogs/qfilesystemmodel.cpp
          // Note that we match the behavior of entryList and not QFileInfo on this and this
          // incompatibility won't be fixed until Qt 5 at least
          bool isDot    = (node->fileName == QLatin1String("."));
          bool isDotDot = (node->fileName == QLatin1String(".."));
          if (   (hideHidden && !(isDot || isDotDot) && node->isHidden())
              || (hideSystem && node->isSystem())
              || (hideDirs && node->isDir())
              || (hideFiles && node->isFile())
              || (hideSymlinks && node->isSymLink())
              || (hideReadable && node->isReadable())
              || (hideWritable && node->isWritable())
              || (hideExecutable && node->isExecutable())
              || (hideDot && isDot)
              || (hideDotDot && isDotDot))
              return false;
      
      qtbase/src/widgets/dialogs/qfilesystemmodel.h
          bool rmdir(const QModelIndex &index) const; // ### Qt5: should not be const
      
      qtbase/src/widgets/dialogs/qfilesystemmodel_p.h
          // ### Qt 5: resolvedSymLinks goes away
          QHash<QString, QString> resolvedSymLinks;
      

      Any of these that can be handled without breaking source-compatibility should be actioned for Qt 5.0.0. Any others should be removed or changed to Qt 6 to-do's.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
            jasmcdon Jason McDonald (Closed Nokia Identity. Please assign to "macadder" instead) (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes